So...yeah. I looked into all those thing while writing, and I agree it works, and I agree the empty-format thing is a bit gross. Besides scrapping the patch, what's a good alternative? I was tempted to simply build the strbuf ourselves, without path functions, but there's a bit of magic in `git_path_submodule`.

The reason for doing it progressively is to record `gitdir_len`. I guess we could start with `git_path("%s", dirname)`, then subtract out `strlen(dirname)` to get the base length. Of course that behaves oddly if the gitdir is `.` and you have a bunch of leading `/`s in `dirname`. Which _shouldn't_ happen in this code path, I think, but now we're making assumptions about how `cleanup_path` works.

Another option is to just stick with the two buffers. To be fair, the "hey, we can use one buffer for this!" optimization is potentially a bit confusing. We could simply leave the `refname` buffer as it is in the existing code, and use a separate `path` buffer (which we also build up progressively).


По идее выполнение жабоскрипта не должно блокировать загрузку, а значить куклоскрипт будет частичео выполняться параллельно с закрузкой страницы, что визуально ускорит его работу.


无意间看到你的repo,觉得这个地方写的不是很专业，这个地方应该是发一个action然后广播到store里面进行处理的，这个地方只是添加一个listener


w8 a sec, in this commit we have:
$this->getCMSLink((int)$params['id_cms'], null, null, (int)$id_lang);

ssl = null, I am going to getBaseLink...

static $force_ssl = null;
		if ($ssl === null)
		{
			if ($force_ssl === null)
				$force_ssl = (Configuration::get('PS_SSL_ENABLED') && Configuration::get('PS_SSL_ENABLED_EVERYWHERE'));
			$ssl = $force_ssl;
		}

now $ssl = $force_ssl so true or false - it can be null because config will not return null, right?

so for this code:
if ($relative_protocol)
			$base = '//'.($ssl && $this->ssl_enable ? $shop->domain_ssl : $shop->domain);
		else
			$base = (($ssl && $this->ssl_enable) ? 'https://'.$shop->domain_ssl : 'http://'.$shop->domain);

ssl & ssl_enable will be true if ssl is enabled and forced.

I don't know where you see the problem


@ZiZuu-store Yep, it's just the way I'm used to write that kind of thing :) I like when it's explicit.


Lol, you me at github,,,,,,


Nice!


Thanks for reporting :)
Fix will go live with our next deployment.


newer to older, right ?


:cry:  去掉激活码后我会将其合并


Looks good to me - this is a good start. Thanks @rjschwei


saw approval from zeeshanlakhani
at https://github.com/basho/riak_ensemble/commit/179c7723eba2a97380348fbf3cbf4b633704fdbb


merging basho/riak_ensemble/dtr/forward_merge_2.0 = 179c7723 into borshop-integration-78-dtr/forward_merge_2.0


basho/riak_ensemble/dtr/forward_merge_2.0 = 179c7723 merged ok, testing candidate = bca3481e


I like the idea of seeing what value was calculated in case there was a problem.  If the value gets altered because it is bad then a corresponding message is also printed to state either the new value, or that the default is to be used.  If we move the asynPrint down then we'll never know what the originally calculated value was.


I'm adapting existing code. The second argument is a string, and in this case there is a macro for that string.


Can we do the `from __future__ import print_function`?


fixed the syntax errors in b8b0a6a3184c6b2d8f3a7138c814d4699996e2d1. I forgot to build and test before I committed...


@jnealb Log a defect for tracking it and mention it in Github PR link.


look at how we do it in boost simd


True, `folder_id` doesn't have to be converted to `str`


Purpose

This allows users to be able to get log information. This will allow more interesting
use cases for the API. Also this will be necessary if we want to be able to use the
v2 api for the project summary page.


Sorry for line-endings.


Ok, you are right, so what the utility of the $ssl var ? Only $force_ssl is usefull


Also, whenever it goes to the db, you should use `unicode` instead of `str` (even though it doesn't really matter here since it's plain ascii in any case).


Everything works fine.
https://github.com/addyosmani/critical/releases/tag/v0.6.0


what's the added value? couldn't we use just
`Figaro.require_keys(YAML.load_file('config/application.example.yml').keys)` in an initializer?
Is it a wrapper for this call, right?


If the debug folder is not created, an Monolog exception is thrown that the specified debug path was not found


```
0: 受け取らない
1: 受け取る
```
の 2 値切り分けだけでなく
```
2: 記事投稿があった時だけメール送信する
3: コメント投稿があった時だけメール送信をする
```
といった変則的 多値切り分け
も発生しうることを見込んで
state って命名にしたんですよね?

であれば、コメントにその旨を書いてください
ぱっと見、```flag``` の命名でもいいんじゃないか、って思ってしまいます


wutt


@valib I'm sorry but I have no video enabled renderer, so I can't test that. I haven't been able to find a useful video software renderer to use for testing either, Windows Media Player is practicly disabled in it's renderer profile since Video, Audio and Image all are set to ```false```. I'm guessing it's a good reason for that, knowing Microsoft it's probably not very compatible with the standard. VLC's DLNA implementation also seems extremely buggy to me, and most of the time it won't even find the server.

That said, I think all these settings (start delay, minimum buffer size, ```FastStart```) should be reviewed so it's logical and clear also to the user who configures it. What needs to be on the renderer (probably only ```FastStart```), and what needs to be in the configuration and GUI? Imo all hardcoded values is a no-no and should be avoided if at all possible. Hardcoded defaults are ok though, as long as they are documented. As I see it, the need for buffering should depend on the server, not the renderer, as only a very weak server will need a head start. Normally, the buffer will probably quickly fill up even if you start with virtually no buffer at all.

These renderers are currently configured with ```FastStart```:

```
BlackBerry-PlayBook-KalemSoftMP.conf
LG-LM620.conf
LG-LM660.conf
LG-LS5700.conf
Netgear-NeoTV.conf
Panasonic-DMPBDT.conf
Panasonic-DMPBDT220.conf
Panasonic-DMPBDT360.conf
Panasonic-SCBTT.conf
Panasonic-Viera.conf
Panasonic-VieraAS600E.conf
Panasonic-VieraAS650.conf
Panasonic-VieraET60.conf
Panasonic-VieraS60.conf
Panasonic-VieraTHPU30Z.conf
Panasonic-VieraTXL32V10E.conf
Panasonic-VieraVT60.conf
Philips-AureaAndNetTV.conf
Philips-PFL.conf
Samsung-GalaxyS5.conf
Samsung-HTE3.conf
Samsung-SMTG7400.conf
Sony-HomeTheatreSystem.conf
Sony-PlayStation3.conf
Sony-PlayStationVita.conf
Telstra-Tbox.conf
```

The big question is how many of these that genuinly have a short timeout and rely on a fast start, and how many are configured like that simply because the configurer don't like to wait. I'm not even sure the ```FastStart``` is needed at all if you could configure the buffering delay on the server. If you have a server that need a certain head start to cope, and the renderer can't wait that long so we start too early, you will end up with a jerky or broken stream anyway. If you have a weak server and an impatient renderer your stuck anyway, if not - you should just adjust the start delay/buffer low enough.

In either case, I think the settings should be exposed in the GUI and documented in ```DefaultRenderer.conf```.


To run the tests as non-root I had to prepend `sudo` to both `sed` and `systemctl` here.
(`sudo` is also used elsewhere in this script.)


closes #8


remove this?


also this.


extra empty space between members and operating.


remove >>>>


IMO the path of least resistance is the first approach you have chosen (before "fixing" it, after I moaned ^^).

For writing concise specs I could imagine to want the "ignore `nil` when mocking" behaviour more often.
But until someone needs it the next time we should probably keep it simple.

Regarding `mock_model`: I actually did not know about `mock_model` for some time and I still need to figure out how it plays along with `FactoryGirl`.
I.e. whether we should use them mutually exclusive or side-by-side; or: In which situation to use which of both...


You can use $ssl = false in getCmsLink so force_ssl will not work


@enricocavalli this needs some reworking.
could you please make it to pass travis tests?


sqlite3 と mysql だと check 制約できない、PostgreSQL は使える
色々あった挙句、L6 - L24 コメントアウトになったんですよね?
ココはコメントアウトして、そちらのできない旨書いておいてください
(PostgreSQL使うかどうかは別として)


#### :warning: Nitpick
I would recommend you to be more careful with the indentation of your code, because you apply it well in some parts of your code but in other parts you stop indenting.


Great! The hint that it implements a bit-wise vertical binary counter was vital. The two lines "ct0 = ..." and "ct1 = ..." didn't make sense to me at all. The rest is clear.


あ、あと check 制約をする意図も


You're right... my fault.


Hum, I will use both, I think so.

And after a review, I erased the "OnUnitStateChanged" function, but I keep the RegisterEventHandler.

I known, I should resist to player request. I created this last alpha to quickly.

Well I will propose a fix for that.


WHITESPACE!!!!!!!!


Is it possible to move the requirement as part of the library dependencies ? Basically I want the code to fail really early since the signatures won't work without the sha256 algo and it's central to the library.


its not a working plugin, i'm developing it too, but its easier to install it everytime from git.

any help appriciated :)


I guess you forgot to remove this empty `if` statement.


model のパラメーターにこういったパラメーター渡ってきますか?
テスト都合が過ぎませんかね?


aquí también lo del `section`


@gRoussac what do you think?


@tcrsavage do we need this here? This function is called from `toc_markup_type_nested()` and `$headers` are already prepared there.


saw approval from zeeshanlakhani
at https://github.com/basho/riak_core/commit/99f9c43e6eaf6414bcad667debea61f183e3c57f


merging basho/riak_core/dtr/forward_merge_2.0 = 99f9c43e into borshop-integration-750-dtr/forward_merge_2.0


basho/riak_core/dtr/forward_merge_2.0 = 99f9c43e merged ok, testing candidate = 91d5df6d


Done


Done.


Hey Jun,

Thanks for the fast turn around.  Although this solution doesn't solve the issue.  It seems throwing the error causes the scroll bar container to not act as expected with angular.  It seems on testing that the only thing that suppresses the issue is to not throw an error and to not hit a null exception.  The original solution is still working for me that I comited. Let me know if it's possible to bring that in or if you have any other ideas.

Thanks,

Adam


all tests pass:
success: http://buildbot.bos1/builders/test-riak_ensemble/builds/218


TeamCity OsmSharp :: GitHub - OsmSharp [Build 1038](http://build.osmsharp.com:8080/viewLog.html?buildId=5054&buildTypeId=bt16) is now running


#### :warning: Nitpick
In the Udacity styleguide is recommended to separate selectors in a newline, instead of
```css
h1, h2 {
    font-size: 40px;
}
```
you should use
```css
h1,
h2 {
    font-size: 40px;
}
```


[これ](https://github.com/ymatsukawa/custom_lodge/commit/9c30c13776de963ce406eef9a914ad32e692fddf#commitcomment-12041366) と同様です


Do you have a suggestion on where to put it?


Not really.. maybe an attachments.js?


.gitignore って ~/.bashrc に逃がすことってできませんでしたっけ?


Addressed in #3626.


Thanks @hkmoon, unfortunately the mapback model is not enough to test. Can you also check the other transformation models that were computed? Thanks so much, Stephan


what have you done so far?

Have you connected to a port already?


#### :warning: Nitpick
A good practice is to leave a space between selectors and brackets:
```css
img {
}
```


No problem. ;)


This is our change where we added a custom class for our `Body` object.


@NobodysNightmare then go write Python ;-)

I've probably become too accustomed to Ruby's implicitness – but this particular example just seems like extra noise. I  can understand that some people might find it more readable.


LiveCode 8 isn't intended to be built on iPhoneSimulator 5.1.


I find it pretty confusing that this returns an object of type `IBadge`, but the `description` is actually the assignment description and not the badge description (which also exists!).


@NobodysNightmare yep, you're right.


looks good to me


Yeah, I thought of something along the same lines to your `GIT_PATH_DEFINE_FUNCTIONS`, but didn't work it out fully. However, that approach doesn't _quite_ work, because we are really doing two things: generating implementations for these globals, and providing a macro for files to make their own file-local helpers. If we accept that all of them will be globals (and that new ones, even with only a single caller, will just get added to `path.h`), then that simplifies things a bit, and we can use that approach.

As far as the efficiency goes, I don't think it's a big issue either way. We have `O(# of source files)` calls to `git_path`, which might as well be a constant. My main concern is the readability and maintainibility. A bunch of same-named possibly-unused static functions in every translation unit isn't exactly elegant. ;)

I waffled on the full-name thing. I like enforcing the naming convention in the macro, but I agree that grep-ability is convenient. I'm tempted to even write out the declarations long-hand in the header file, and make the implementation macro purely a private thing in `path.c`.  That's even less magical.

I was also sorely tempted to do the code-generation thing in perl, a la `generate-cmdlist.perl`. That's magical, too, but the resulting code is very straightforward.

I'll take a look at moving all of these path functions into `path.h` as a preparatory step.

To be perfectly honest, I'm not sure exactly what `update_common_dir` is doing. I saw it and shielded my eyes immediately. Reading c7b3a3d2fe2688a30ddb8d516ed000eeda13c24e makes me cringe.


.rspec って どこかに共通化できないですかね?


return getSwaggerType(p) + "<" + getTypeDeclaration(inner) + ">"; //as an alternative but duplicate code from JavaClientCodegen


軽く調べてください


:-) I suppose I should have put it under the Backward Incompatility secion.


Looking forward to your help! :-D


this should be a runtime dependency.


Yeah, it would probably be nicer, but I figured I should just do one refactor at a time.


What happens if this returns NULL?


The class should be generated to SRC and it should be Xtend code.


maybe just call self._parse_response(response)?


Then change the message from "Setting istorek=" to "Calculated istorek="...


Meh, you should pass those `browser` and `compress` option to cssnext, not postcss


#### :white_check_mark: Awesome
Good use of "viewport". Your project looks good on different devices.


To be honest, I'm confused about how leap seconds are manged wrt NTP myself.  I found this https://www.eecis.udel.edu/~mills/leap.html that holds some promise to provide the answers I seek, but I have not digested it completely. What bothers me is that without any leap second processing in my implementation, the NTP server seems to provide what the Windows NTP client expects.


I am confused with the implementation of Splay Tree, specially in the function splay and remove interface. And I think their implementation doesn't match the description in the textbook in P161 and P164. Please contact me at your convenience.
However, this confusion may caused by my misunderstanding of the Splay tree, but I am not sure...


creating


I'm not sure what the best approach is here -- iPhoneSimulator 5.1 probably shouldn't be in the list, but I'm not sure if it's a good idea to have to continually maintain this difference between LC7 & LC8 branches. @livecodesebastien?


How so?  `get_groups_size` returns the total number of results matching the group query, and `group_es_call` performs the groups elasticsearch query.


処理内容と命名が不一致な感覚がします


I'm sorry, I marked the wrong bug# in this commit log. It actually fixes https://bugzilla.xamarin.com/show_bug.cgi?id=31196


I think we have to catch the case where `getUser` does not return the user data.


nice job!


We need both '-lm' and '-llog'.


Not really sure type args should be placed under reference expression.
<img width="407" alt="2015-07-07 18 07 37" src="https://cloud.githubusercontent.com/assets/2060943/8549376/1c3ded2a-24d3-11e5-96ba-545ddb78e3ee.png">


Update the checkout fields form to let developers customize even the field "Ship to a different address?"


congrats, you just created an NPE. http://docs.oracle.com/javase/7/docs/api/java/util/Locale.html#forLanguageTag(java.lang.String)


s/urlib2/urllib2/

please test this with python2.


TeamCity OsmSharp :: GitHub - OsmSharp [Build 4.4.0.1038](http://build.osmsharp.com:8080/viewLog.html?buildId=5054&buildTypeId=bt16) outcome was **SUCCESS**
Summary: Tests passed: 542 Build time: 0:0:0


Your proposal is missing the environments.


I really like how you made all the smaller processes that could be repeated their own functions, but I feel as if the main(args) should only contain the processes you want to run, being the arguments and the function that you want to run. Adding the large function to the main(args) makes it more difficult to copy and reuse the same format for making other programs runnable through the command line.


@serverfire I've been using your blog on a project that I'm working on a the moment. When I'm finished with that I'd be happy to work on it. What ideas do you have?


should be abstract


really good info here, but i think this belongs below the main usage area.


These are internal URLs and aren't useful to others using the role... You'll either need to variable them out or you could just inject this custom template via another role/playbook.


Just to be sure: Is the order of AR statements relevant?

I could imagine a difference between: `limit(count).where(condition)` and `where(condition).limit(count)`.

I could imagine that the first example will limit to the desired count and **afterwards** apply the `where` to those few results.

What we want is to select using the `where` clause, but only take the first `count` results (according to the order).

**tl; dr:** Does the order of AR statements (`limit`, `where`, `order`) matter?


@fivetanley was this just an experiment or.. It seems to have gone stale


```[1, 2, 3,]``` のように最後の要素のあとに ```,``` つけられるので付けておいてください

今更ながら、よう考えたね、この validation


all tests pass:
success: http://buildbot.bos1/builders/test-riak_core/builds/1605


Default this to 'local.conf.j2', and bring the old template back. That way the default is to use the provided template, but you can still inject your own.


I really like how you did the Qscore reading here. Making it position based made it easier to convert to the older version of Illumina without having to rewrite everything. A lot better than what I did.


I noticed that - @liach pointed it out.

Thing is, as soon as I extend BlockBreakable, my IDE craps out and tells me that "there isn't a default constructor".

Does Mixin have a workaround for this or do I have to provide a null constructor myself?


fast-forwarding 2.1 to borshop-integration-78-dtr/forward_merge_2.0 = bca3481e


2번째 인자부터 콜백의 인자로 전달 된다.
1번째 인자는 삭제 확인 메세지에 이름이나 정보로 사용됨.


I dont think it can return nil. If the id exists, the contact must exist
too, no?

On Tue, Jul 7, 2015 at 10:23 AM, Rodrigo B. de Oliveira <
notifications@github.com> wrote:

> This transducer can return nil, compose with filter? (comp (map (...))
> (filter some?))
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/sneerteam/sneer/commit/e559f9b70846ec79e073ec2826e9dd388df80ce9#commitcomment-12039064>
> .
>

--
Valeu, Klaus.


You should test passing null to the language param. Code for robustness, do not expect everybody will be calling the API properly.


Would it be possible to release a beta npm package of karma-jspm based on this commit? We're using jspm@beta and would appreciate compatible of karma-jspm.


Yes yes I agree, I am just saying that I think (without actually spending much thought on it) that this wasn't a good design decision. And I might add that I suspect that there are *many* packages that have the same bug.


![](http://images.akamai.steamusercontent.com/ugc/686020718149174139/35E9DE941E46EA4A62D56462D1847537F4F3FEAA/)
终于在游戏里见到这个东西了


Just provide any constructor so that it compiles, all constructors are ignored by mixin anyway.


Very interesting way to do that lol


Download and Launch I see as somewhat separate responsibilities.   It's preferable that the launcher here just be responsible for "launch" and some other class take care of "download" duties.


Does this add anything that the url doesn't? What if the setup instructions chnage?


In your commit message, you mean this module is redundant to batchdelete, not delimages, right?


typo


If this is easy to do, suggested name change:
get_unfulfilled_pre_requisite_milestones or something that uses the same terminology for milestones.


test


配列最後のカンマ付け、お願いします


Thanks for the catch. Actually it was tested with Python2. But it I didn't notice it was using the installed library. fixed.


Hah!


fixed


@bermudezjd I don't think that I need to store sessions for my tool, so should I still add the REDIS settings?


@NobodysNightmare

> Was that an automatic removal by rubocop?

it sure does!


Let's see what's going on!!!


My bad, thanks!


The name should be Abstract<LANGUAGE>ColorSettingsPage, but I'm not sure it is worth having two classes here...


OK I am fixing it


iSrc++ should be enough - no need to deref here right?


Coolio - thanks! :)


`YAML.load_file('config/application.example.yml')[Rails.env].keys` ?


@NobodysNightmare quite right! _bad cop_


lol


This is the "mascot" not the "logo". Did you look around in the existing images? Are you sure it wasn't there? I'll merge this, but please rename it to "boris" or "mascot", not "logo".


Whoops, I mean to put #2191 in the commit message.


thanks!


`total` should be the grand total, no matter what page of results you're on, so I do think that should go at the top.  This logic is pretty difficult to reason about in general, which is why I pulled it out to test in the first place.  I don't think I'm really following what you're saying, but if you've got an edge-case in mind which this might not handle correctly, it'd be great to include that in a test (whether or not this already passes it).

Can you describe the test you had in mind?  Or add a commit to this PR including it, if you're feeling ambitious.


yes, it is more correct to use put, however it will work when sending a post, I implemented and tested it


Released as 1.51.


@XeeD @steida just pass custom `getLocaleFromRequest` that receives current request (`req`) and available locales to load as a list. If you return undefined, `defaultLocale` will be used.


Initial set-up


It isn't cool to break the existing tutorials. How does this impact them?


And this actually works


Cool. I'm assuming all that's required here is to remove from the ```deployUsableIosSdk``` list.


@NobodysNightmare removed in 75cecdb21aed1106a1e61f1563496cb170626809


> (unless, of course, it's not being tested)

I can imagine that it is untested very well...

I am not sure if I understand correctly when you will reintroduce this: Will you reintroduce it with Rails 4.1 or with Rails 4.2?


Imo it's probably better to have this code in a gem than in every project :)


Ya. I changed it to response = self._parse_response(response, None)


You can do:
```
Tasks.find() // get all tasks, an empty query returns everything
.forEach(  //iterate over each document found
function(task){
  Tasks.remove(task._id); // remove task using _id key of task document
})
```


Whoops. That was my mistake. Should be fixed next push.


Why is this a separate script file? The content seems redundant.


@christopho I noticed the problem after my last system update (with `Qt 5.4.2` and `Mate 1.10.0`), can you confirm that's still ok for the Windows version?


Good point.
I will send you a list of gist ids for this.


### Threat Models


They didn't exist in the implementation, but now that I'm looking back at it, the idea should be to match the implementation to the docs, not the docs to the implementation. I'll correct it in the next commit, unless we do decide to get rid of it.


Is this commit right? seems like a lot of changes for http://manage.dimagi.com/?169440 (and also odd ones like introducing unused imports).

just wondering if it might be a bad merge or something


There is still the same issue with the iPad 1, which can still get the applications uploaded on the appStore - provided that the minimum version is 5.1.
I think that we can drop 5.1 once the iPad 1 can't install apps from the AppStore anymore - but correct me if I'm wrong


@riteshchandawar @lukechurch This is tracked with MAGN-7845.


*they


lgtm


@NobodysNightmare Rails 4.1 is the plan (hence adding `TODO`/`FIXME`comments for easy grepping)


field_values って命名は、使うとしても、改めて見返すとイマイチ


You @lucas-jones are a scumbag.


Good doctest with the format. Since the function is to add an error, it will do so 100% of the time.


What do you think about this:

https://github.com/NielsH/RaidCore/tree/fix_maelstrom


According to magnum, the above error messages can be ignored.
The errors that need to be fixed are indivifual failing / segfaulting OpenCL format self tests:
https://github.com/magnumripper/JohnTheRipper/issues/1499
https://github.com/magnumripper/JohnTheRipper/issues/1504
https://github.com/magnumripper/JohnTheRipper/issues/1505
https://github.com/magnumripper/JohnTheRipper/issues/1506
https://github.com/magnumripper/JohnTheRipper/issues/1507
https://github.com/magnumripper/JohnTheRipper/issues/1508
https://github.com/magnumripper/JohnTheRipper/issues/1509


that one needs to be updated as well


thanks @DarrenN sorry to trouble you :)


The list option is used in the `mscs.completion` bash completion script, and I think that this change would break that script.  I didn't add `ls` to the documentation because it was synonymous with `list` like a number of other commands, e.g. `usage/help`, `status/show`, `map/overviewer`, etc.

What about adding this information to the `status/show` command instead?  The output from that command isn't parsed by anything that I am aware of (it might break some user made code I guess, but I'm not worried about that).

Alternatively, the `mscs.completion` script could be fixed to work with this change.


Once you have that paste url you should throw away the rest.
As you can see on https://gerrit.wikimedia.org/r/#/c/223308/ the message is the review rendering the paste url unimportant.

Expected: output = "Tests passed.\n<url>"


issue #9; EntityMode/Model/Textures on sourcecode can be loaded. How to use in dev will change! Modifying jar for develop will no longer be needed.
After release, I make the page about it on wiki.


Hm, this first line still reads weird to me.

Something like "Despite [evidence](https://www.aclu.org/report/liberty-monitor-all-how-large-scale-us-surveillance-harming-journalism-law-and-american?redirect=liberty-monitor-all) to the contrary, some journalists believe that only reporters working directly with national security stories should worry about NSA surveillance. Edward Snowden and Chelsea Manning are two recent high-profile examples of leakers, but the truth is that journalists working on a whole host of different story types should be thinking about their digital security.


Hm of course that's only for the minimum version - I suppose that as @livecodeali said, we can remove 5.1 from the usable SDKs (and I guess that's what you meant, @peter-b ).
We might want to change that from develop-6.7 though


Is the if test reversed here?  Should we add to the download list if checkForPresence returned true?


@samlecuyer not a problem! Happy to see chapters keeping an eye on things.


Our own custom finder method `User.find_by_login` is case-insensitive, whereas `User.find_by(login` is not. @oliverguenther you're right that I probably should have split the revert of parts of 74228b5 into a separate commit. This would entail rebasing the whole branch though, something I'd prefer not to do now.


Adding the **woocommerce_customer_save_address** action doesn't seem to work.
Does it have to be registered as an order status first?
```php
// Add a custom email action to the list of emails WooCommerce should load
add_filter( 'woocommerce_email_actions', 'add_customer_address_change_woocommerce_email_actions' );
function add_customer_address_change_woocommerce_email_actions( $email_actions ) {
  $actions[] = 'woocommerce_customer_save_address';
  return $actions;
}
```


thx.
It looks like _parse_response() doesn't use the all_ops param anymore, can you clean it up while you're here?


fopen() and fclose() are not needed. ZipArchive will do the necessary file operations.


i mean its by kufikugel who made that change ...you should be ading him as credit.
thanks


Проще новую вкладку открыть.


Should this be here? Might as well use a separated module.


Так ведь он два раза выполняется теперь.
Второй раз в `Initialization`


So this makes it such that you can only link to one backend?
If this is the case then we might need to reconsider. A common operation is array on cpu, array on gpu, etc.


Yeah, I have a module for it. Just haven't had time to publish it yet.


You have modules for everything :p


yay!

On Tue, 7 Jul 2015 at 15:54 Ben Zörb <notifications@github.com> wrote:

> Everything works fine.
> https://github.com/addyosmani/critical/releases/tag/v0.6.0
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/addyosmani/critical/commit/0f57910af8842737b869a89fca2725874fcaf0a2#commitcomment-12041206>
> .
>


good!


harvester/


Pull request updated. Thanks PreVaDu!


@mrydengren good idea, we'll do that now. I've been hesitant to cut the actual 2.0.0 until we are more confident with the stability.


> They should be moved to lib/LaTeXML/resources/Profiles/, just removing the blib prefix.

@dginev I've created a corresponding branch https://github.com/KWARC/LaTeXML/tree/move_profiles, but when I create a pull-request it tries to make one on for https://github.com/brucemiller/LaTeXML.
Shall I simply merge the move_profiles branch to master?


Heh yeah. I messed around with github releases a bit and I couldn't really figure out how I could implement it under my current updater. The biggest issue is that github isn't really set up like a file server (and I don't think they want people using it like that). I tried to get direct download links to work but it kept failing so I decided to just get something easy. The Mediafire uploader is super simple and I was really just trying to do as little work as possible to get it off SF.


sure. cleared


This can also be done during zipCreate(). The file can stay open until the script added the last file. I also just stumbled upon the method addFromString() in the documentation. It takes a string as the file content, which makes the temporary file stuff obsolete.


@kevinpschaaf When using this commit and the current Polymer master and master for all paper-elements, type and item are undefined when creating a paper-slider with snaps set to true

          <paper-slider flex immediate-value="{{block.padding}}" value="{{block.padding}}" min="0" max="10" step="1" snaps></paper-slider>


For the first question: I do have an idea how to solve this. I'll update the PR today. If it won't solve you issue will look into another approaches.

> It is under Apache 2.0 license - may we use this?

Have no idea. Need to check.


`title` still not showing up but the opacity change now works with https://github.com/mtennekes/tmap/commit/893b08cf76d31f16a20c234678fe6d676131676b


Closes #9


This commit title should read "Add lzop to the bootstrap list"


Much appreciated. :+1:


@NobodysNightmare this is actually a great example: despite chaining AR query methods in a different order (and handling arguments slightly different), `News.latest` and `News.latest_for` both generate exactly the same SQL:

```
> News.latest.to_sql
=> "SELECT  \"news\".* FROM \"news\"  WHERE (((projects.status=1 AND projects.id IN (SELECT em.project_id FROM enabled_modules em WHERE em.name='news')) AND (projects.is_public = 't')))  ORDER BY news.created_on DESC LIMIT 5"
> News.latest_for(User.current).to_sql
=> "SELECT  \"news\".* FROM \"news\"  WHERE (((projects.status=1 AND projects.id IN (SELECT em.project_id FROM enabled_modules em WHERE em.name='news')) AND (projects.is_public = 't')))  ORDER BY news.created_on DESC LIMIT 5"
```

It would make sense to deprecate one of these methods.


I mean that the styles added for an individual page should come *after* ones provided by a framework such that they override the framework settings.


Yes, we maybe ought to mail the list. If people are installing from
scratch, this is no issue, as the dependency is now listed in both setup.py
and requirements.txt.

There are a lot of overly strict string checks which should be refactored
like this for Python 2/3 compatibility.

On Tue, Jul 7, 2015 at 6:08 AM, Trevor Bača <notifications@github.com>
wrote:

> So the addition to setup.py means that we intend the six module as a new
> dependency?
>
> I had to use sudo pip install --upgrade six for everything to work with
> this one. So if we intend the new dependency we should probably mail the
> list and let everyone know.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Abjad/abjad/commit/fc7ce993c0e0d8d90a1c2ebf760e75fcb2926b58#commitcomment-12038778>
> .
>

--
Josiah Wolf Oberholtzer, PhD
http://josiahwolfoberholtzer.com


Let me know the module name and I'll use it in here.


@telliott22  would be good to make better theme for it and few more options in panel like tags and category for blog , that would be easy with laravepanel .


file_ptr -> avoid copying this one.


I think this can be removed and the calls to it replaced with `$report->get_time_spent`.


FYI, this is all discussed in the comment thread for the pull request: https://github.com/Abjad/abjad/pull/467


> I'm also surprised that doing preloads without explicitly adding the .sty extension works, didn't know LaTeXML auto-guesses the extension (does it?)

all stex*.opt modules omit the .sty extension (even the older ones that I did not move).


@kevinpschaaf Because here paper-slider sets an array with a length, but no entries

https://github.com/PolymerElements/paper-slider/blob/master/paper-slider.html#L424


Autoprefixer can add the vendor prefixes automatically, so that you don't have to deal with it ;-) https://github.com/postcss/autoprefixer. It seems there is no Meteor package yet that directly works with CSS. There are packages for Less and Stylus though.


Feel free to merge your branch!


What do you mean by "lower down"? I moved this up to clear up some ambiguity about how these pieces interact, but I don't think location is lexically significant because `currentPlotMover` won't be evaluated in the local scope until the function is called.

So, then, I think it's:
```
var currentPlotMover;
function onSaveBefore(data) {
    if (currentPlotMover) {
        currentPlotMover.onSaveBefore(data);
    }
}
```
vs
```
 var currentPlotMover = {onSaveBefore: _.noop};
 function onSaveBefore(data) {
     currentPlotMover.onSaveBefore(data);
}
```
?

I prefer the latter because it is shorter, and exposes more information about the interface that `currentPlotMover` is expected to implement.


`IsVideoOSDActive` and `IsMusicOSDActive`?


Why are you not able to [replace those calls](https://github.com/myabc/openproject/commit/355e7e0ff4a6c30e864eed19ae1c9dc69fbd5971)?


把我之前讲的4种动词补充上去


Sorry about asking here, but no other way to contact with anyone, no pm/s or emails is provided.
I want to know if teensy can provide support for s29gl256 one more pin of data a23 needed, nor norway can support it at soft edit with using `B7` pin for A23 ?

Best regards;


Bit of a shame that there's so much duplication between this and the SDM template, but that's not really in the scope of this PR.


Лучше полностью проверять имена окон, а то если борда будет использовать `postMessage`, то скрипт будет каждый раз пытаться проверять удалённые посты.
Вот так:
```js
if(typeof e.data === 'string') {
    switch(e.data.substr(0, 15)) {
    case 'de-iframe-pform':
        checkUpload($DOM(e.data.substr(15)));
        $q('iframe[name="de-iframe-pform"]', doc).src = 'about:blank';
        return;
    case 'de-iframe-dform':
        checkDelete($DOM(e.data.substr(15)));
        $q('iframe[name="de-iframe-dform"]', doc).src = 'about:blank';
        return;
}
```


Nit-picking here as I know you will be working on fixing this test - I think it is always good idea to specify why the test was ignored. In case it was forgotten and stays ignored for longer time, it is then much easier for someone else to determine why it was ignored (e.g. failing always, failing in CI because of ..., failing on some specific JDK).


GSRendererHW.cpp
No change 1210 line
bool GSRendererHW::OI_SuperMan(GSTexture* rt, GSTexture* ds, GSTextureCache::Source* t)
window build failed


@NobodysNightmare in this case, there a several calls - I think it's tidier to add our own instance method that aliases the class method.


The changes look good, and so do the comments. The tests are all passing now. I am going to merge this.


those conditions can be combined


you are back!!! cool!


that looks better, after your comment I realized it previously did not do much so this makes it a little more clearer and a better use of the function.


`IsShowInfo`?


Looking good. +1


I think you can ignore the [Travis build failure](https://travis-ci.org/simoncozens/sile/builds/69907470) on this one. I think their VM just hiccuped because the same commit [ran fine in my Travis accound](https://travis-ci.org/alerque/sile/builds/69912483).


With this pull request we lose the ability to detect changes in mercurial.. The problem is that `zstyle` seems not to merge the values correctly together. That is the reason why I added a static hook for `vcs_info`.


세상에


@andrewxhill do the PR


What is the advantage of using an Array and joining it over using a String? @seenmyfate


Реально.


put this back in and wrap in isLoggable()


So the $delim_output is predetermined by us whether or not it is a delimited string or JSON I'm guessing? I'd imagine whenever we hook this into the front end and pick one of either delimiters or JSON we can remove this and the if statements checking for this. I do like this idea though, it leaves flexibility for the front end.


Regress back to 99.0.3


Thanks for adding documentation to the fields in this file. Please document the fields in `Status` as well.


Ahhhhhhhhhhhhhhhhhhhhhhhh


sorry, It seems that I forgot to force push the final changes so that it didn't reach the PR.  I will add a fix PR for this in a instant


Do you know why lost is a dev-dependency vs a dependency? I never understood the difference


I don't get it... why do you need to validate the default user?

If there is a default user configured, and no user/password was passed, all we need is to return the default user.. no need to validate its password.

The password may not be actually available.. Think of LDAP or other providers...

Unless you have a real failing usecase for this, this should be closed / nor merged.


@NobodysNightmare If you're feeling adventurous, you can also try calling `User.latest.arel` and  `User.latest_for(User.current).arel` and compare the AST that Arel builds.


Many thanks for taking care, @Jalle19. I and others obviously did not test stuff on Windows. Worked fine on Linux, using no other platform atm.... Sorry for the mess.


The GUI now looks like the screen shot below. However, the Calibration settings, sectors and filters are all currentl linked to the same Show/Hide button.
![callibration filters](https://cloud.githubusercontent.com/assets/13198775/8550478/aacb4426-24c8-11e5-9eed-c5f07659f32d.png)


fixed.


This line effects a few things in regards to the accordion component. It assumes that the user will have downloaded/copied the list styles from the pattern library. Without having done that, this class will not work. Since this component is meant to be a stand alone item without having other library dependencies, I am not sure it works well to use this class here.


Nice! I thought you were only using these completions later...


Looks good, can no longer reproduce the log message about EventEmitter close leaks.


when doesn't it?


Hello SESupply,

Thank you for your comment.
To make it work in ESPArduino / Arduino 8266, one must change this library to an "arduino library" and then test it.

As mentioned in issue #1 I really would like to port it to Arduino. But right now I do not have enough spare time.


이거 #NO-ISSUE에서는 원래대로 고쳐져 있는데 Developer에 반영이 안되네요.


기계 학습은.. 결국 인공지능 SW 이고.

어떠한 데이터를 기반으로도 사람이 분석한것 처럼 자동 분석하는게 목표이지만,
현재 대부분의 경우는.. 시스템의 Log 파일을 분석하는 정도임.

Readme 에 더 써 놓도록 하겠음.

2015년 7월 7일 오전 1:10, hsji123 <notifications@github.com>님이 작성:

> 핵심 기술을 정한다면 (2)를 중점으로 해보고 싶습니다.
> 그런데 개발에 대해서 어떤 센서 데이터를 가져오고 어떤 API 가져와야할지 방향을 못잡고있습니다.
> 여기에 대해서 도움 부탁드립니다.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jeonghoonkang/keti-1/commit/fcab9caa146fc8c13f6005886ac916e0a7cd1ff5#commitcomment-12020030>
> .
>


official solution 总是比我屌啊！
思路特别屌，some every 用的出神入化
```javascript
module.exports = function checkUsersValid(goodUsers) {
  return function allUsersValid(submittedUsers) {
    return submittedUsers.every(function(submittedUser) {
      return goodUsers.some(function(goodUser) {
        return goodUser.id === submittedUser.id;
      });
    });
  };
};
```


cblach thanks for your help !!! We manage to install everything in the beaglebone
Now we have in the /usr/local/include all the four files install (hopefully)

Can you test how we can run out a test ? In our test rig we have two Powerline GP with SLAC .. Is that a problem ? because you describe the bench rig the one to be with SLAC and the other one without SLAC ?

In addition, can we run a test without the test rig in order to check if the beagle bone is working properly ?


Right, this make sense. I will fix it once I get home.

I was just worried that out tree is too flat:
new A ---[String] "asdf"
Infix expression
  New Template definition
  Reference expression
  Type Args
  Literal

But I guess this is fine.


remove


it should be left in both, but commented out as an option. We actually had someone file bug at one point in RMQ because they use Mercurial or some such


Fixed in 3.1.1


Minor but the "set" prefix seemed preferable to me.


scale cannot be equal to precision?


You can specifically select which branch you are merging against in the upper-left corner of the pull-request interface.


lgtm


Although common, I have never been a fan of using the word "point" in this way because some people read point and think vertex.  I am not sure if the following is any better, but...

>Returns ``True`` if the interior of `other` intersects the interior of `object` and neither the interior nor boundary of `other` intersects the exterior of `object`.


I'd also have code that represents dirty solutions look weird an intentional, lest it be confused as something straightforward. We have a circular dependency that is managed by closing over the outer scope; I'd prefer not to lose track of that detail.


@dginev I've also created a commit that deletes trailing spaces and ensure final newlines. https://github.com/KWARC/LaTeXML/commit/5e09525d2f60bdd1e72813b6e1fb2a6bc2a2f314


Hi,

Sorry for slow response and thank you for your question.

The image optimizer has been released in May 2014 as part of the V3 launch. The optimiser creates cached versions of the resized images on the hard drive, used to supply images that have already been resized. So if the image is in cache, it won't be resized again. On top of that you can control the maximum size and the factor which essentially limits the number of different sizes the resized images can have. In addition the image optimiser does not serve the image larger than the size of the screen.

This is a very basic image optimiser designed as a perk for users of 51Degrees device detector and not intended to compete with other image manipulation solutions.

It has been successfully used by a number of our customers over the past year. For example: it is being actively used by the Loterie Nationale - the national lottery game for Belgium: https://51degrees.com/resources/case-studies/loterie-nationale

There are some open design issues that we are happy to receive feedback and suggestions about. I.e. The cache on disk does not get automatically cleared. We are considering adding a timer task that would clear the Image Cache folder once per day or so. Another issue is that in rare cases the cache path becomes too long and exceeds the maximum system path length. However this problem is not critical as the image optimiser can be easily turned off. We are currently considering alternative caching methods, but the issue is very rare and mostly occurs if the website is located deep in the file system to begin with.


done. https://github.com/PCSX2/pcsx2/pull/637

sorry for the trouble


tagging @JHoloboski


+1 tested on windows.

Also in agreement with @Jalle19 in splitting commit out to fix PVR asap, if this PR is held up by the other commits


I would prefer the PR in this case, so that I review this in peace (it won't take long) and avoid any other sneaking problems.


Did you add anything to your `script.js` file? Not seeing anything.

We don't have our 1:1 meeting until next week but if you want to meet during my "Open Appointment Time" on Thursday I'd be happy to. We can do some pair programming and figure out how to approach this assignment. I have a feeling that once we take the first few steps you'll know where to go from there.

Just let me know on Slack.


I think this should read "Red Hat, Inc. and others" but IANAL.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 571](http://build.prxm.it:8811/viewLog.html?buildId=3403&buildTypeId=Jarvis_DocumentStore_CI) is now running


@tsing , done, thanks for reminding me


@kinglozzer do you want to do some investigation into this?


I would restructure this block.  This will catch the IllegalArg exception thrown just a few lines back.


Could it help to do this instead, until we know exactly? I cannot really see how to determine the exact syntax by now.
```php
        if ($dbn && $serendipity['dbNames']) {
            @mysqli_query($db, "SET NAMES " . $dbn);
            @mysqli_set_charset ( $db, $dbn )
        }
```


(here and below) Please mention the units in the documentation. Or change the variable name to include the units.


This URL is dependant on your environment, so needs to be pulled out to a separate environment file, which isn't managed in Git, but which you DO include a template.

For example, my URL is http://local.test.com/kevin/homework-form-index.php so I'd need to be able to set the URL root.


Nope, that's good justification (kernel documentation). Thanks for adding that. Could you update the commit message to reference the kernel documentation and resubmit with the patch content unchanged? I'll apply that.


This should still throw.


This should be changed to something like:
```
global.expect(function() {
    checker.configure({enforceExistence: {allExcept: 'something invalid'}});
}).to.throws(/accepted value/i);
```


Then how about just print `console.error` and return? I don't have much idea about angular, so could you test it out?

Thanks,


Yes, this is fine. Also please make sure these type args work correct with the related intentions (I mean "Convert to infix expression" and "Convert from infix expression")


Nice fix!


Btw I checked and indeed LaTeXML will default preloads to ```.sty```, so the profiles are using a legitimate shortcut.


right, thx


@beghelli, could you take a look?


COLONS ON LOCATE(HREF)

HELP


I think you can disable SLAC on one of the powerline GPs, you should be able to follow insys manual on how to do so.

You are able to run the program locally without SLAC, yes (just make sure you use the same interface, i.e. wlan0 or eth0). I have a pdf from my master thesis which documents the use:
https://dl.dropboxusercontent.com/u/4875148/MasterThesis_cblach_excerpt.pdf

Note that the -s enables SLAC.


Sure you mean $_GET[$required_fields]? Surely $required_fields is set then you have form data.

Also, you don't seem to be setting it in the header of your file.

Something like this:

session_start();
$required_fields = $_SESSION["required_fields"]; // get my data out of session, and populate my variable

FYI, session_start(); needs to be FIRST THING in the script. So before any output to the browser like a <html> tag etc. So needs to be:

<?php
session_start();
// all my other stuff
?>


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Hey @kevin-chilcott-sift , what do you mean by it doesn't work? How is it not working?

I've made some comments below, how that helps. If not, just let me know and I'll look again for you.


This line actually needs to be:
`define('CANDELA_OUTCOMES_GUID', 'CANDELA_OUTCOMES_GUID');`
because when we first pushed this to production we had 'CANDELA_OUTCOMES_GUID' in all the get/set meta calls initially and there is now production data using it.


Josh, Can you change the name here to jcbremer


`print_r` is mainly for humans, I don't think it should be used when formatting things in html attributes. Just getting rid of it should be fine.
Also, you need to escape this on the way out like on the edit page. so `esc_attr()` again.


@coditect Could you please let me know why is necessary to rotate each frame individually when using GM? im.applyDataFunc() should do the right thing under both IM and GM.


Sorry. I think it was me who broke that.


Now those classes do not violate any rules.


I think it makes more sense for phpunit annotations to be inherited though.


Old Chrome Appearance

![chromeshotchartrender](https://cloud.githubusercontent.com/assets/1585301/8550859/6231f30a-2490-11e5-8c8e-f12517646b6d.png)

New Chrome Appearance

![newchromerender](https://cloud.githubusercontent.com/assets/1585301/8550896/9e9bc30c-2490-11e5-90f0-59b362737362.png)


오늘 수업에서 나온 `defer`! 선견지명 엘리트!
감사합니다(--)(__)


So all these Driver extension replacements are not ideal.  Can we consider adding additional constants to SeLionConstants ? or perhaps use something else besides Selenium CommandLine.find()?


Thanks for getting back @gylaz.

What I mean by "proving" is important to my argument, so let me elaborate a bit. When a `context` is wrapped around something when also applying a `let` or `before` or what have you uniquely inside the `context`, it's more than just a notation, it's an actual behavioral structure that changes every example within it.

By using `subject` and `let` you're able to keep all the code which changes what's being tested, ie, `context`, `subject`, `describe`, and `let` from the code which is checking things about it, ie, `expect` so that it can be "proved" that it's shared across all the examples. The `context` docstrings needs to match the behavior defined in the corresponding `let` and `before`, and the `it` docstring needs to match the corresponding `expect`. Once those are visually verified you're done, you don't need to think about whether the code in each example also matches every `context` docstring above it because that happens automatically simply by it being within them.

I agree though, it's more difficult to read what's going on if you open a file, go to a specific example, and try to work backwards from there only caring about figuring out that single example... Though the examples get so visually lightweight that IMO it's far easier to verify and consume a full `describe` of examples when `let` is used, assuming there are more than a few `let`s being used to spec each `describe`.

Maybe for an organic spec file that's expected to change and the examples will need to either be frequently adjusted and reverified or primarily used by a small team that will be somewhat familiar with it, it could be argued that `let` is more potent, but for a relatively static spec file that's intended to primarily serve to document code usage (code usage in particular since if it was just the docstrings that were being read, neither would make a difference) to those unfamiliar with it, `let` can be confusing? Maybe `let` can seem so good because the one writing it is always familiar with it :P It sure is amazing for TDD, maybe just not quite as good for regression.

And sure, I could take a crack at it.


즐겨찾기 추가했습니다 코드펜! 눈으로 보여지니 참 좋네요.


Also! using `for` loops separate the looping mechanism into one line, and your logic in it's own sections


The CLI programs all log the error already in whatever mechanism they want.  For example, if no arguments are provided, the help message is displayed followed by a list of valid commands.


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Similar to the session_start(); tip below, can't do redirect if data already sent to browser, so <html> tags need to go after this lot. In fact, if there is no output by this script, then doesn't need HTML tags at all, as it will either pass them forwards (success) or back (failure).


Usr died for our Sins! GIANT BUGinism is a real religion! Embrace integration testing! Save your projects, save your life.


didn't get you quite, do mean we should expose both `afcpu` and `af[cuda|opencl]` under same crate/package ?
`arrayfire::cpu::Array` and `arrayfire::gpu::Array` ? and then based on the which module (cpu|gpu) function is called on a particular Array we do processing ?


Cool, if we perhaps record that we're aware this test/functionality was not needed in the commit messages for future reference.


This could be moved below the if check


A better fix for issue #113 based on @AccaliaDeElementia's suggestion


We may want to link to the issues that discuss this, but for now I think just having this present in the FAQ is definitely valuable. LGTM.


:+1: nice exhaustive list of conditions.


Overall, looks good, the only really tricky one is org.killbill.persistent.bus.main.inflight.claimed. Having bigger value may not necessarily be an advantage. I think what we should do here is add a Histogram (https://dropwizard.github.io/metrics/3.1.0/getting-started/#histograms) to understand how much each thread really takes when we throw some load (might even be good a to have a per-thread Histogram) to see if we really use all the threads.


@lukecarbis Why did you decide to remove this? See https://github.com/xwp/wp-widget-customizer/issues/74


Hmmm what do you mean precisely by `makes more sense for phpunit annotations to be inherited`?


@rgladwell actually i did not know the previous reason of this test until now. :S


Not sure we should expose internal objects like this: maybe we should move the `mojoExecutionIsManifestMerger` to `JaywayMavenAndroidProject`?


:+1: nice simple implementation.


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 572](http://build.prxm.it:8811/viewLog.html?buildId=3404&buildTypeId=Jarvis_DocumentStore_CI) is now running


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 571](http://build.prxm.it:8811/viewLog.html?buildId=3403&buildTypeId=Jarvis_DocumentStore_CI) outcome was **SUCCESS**
Summary: Tests passed: 290 Build time: 00:14:25


If you had a method annotated with `@before` then extended the class with that method, before this change, phpunit ignored it.


With the latest MkDocs I hope this will now correctly fail your build.


It took me a while to hunt it down in the source code, they've been so many re-factorings.


Docs at http://php.net/manual/de/mysqli.set-charset.php say:
> This is the preferred way to change the charset. Using mysqli_query() to set it (such as SET NAMES utf8) is not recommended.

Another http://dev.mysql.com/doc/apis-php/en/apis-php-mysqli.set-charset.html

Seems to not be easy to to get answered strictly, though. http://stackoverflow.com/questions/26596294/set-names-vs-mysqli-set-charset-besides-affecting-mysqli-escape-string-are


Since you're already handling the `EEXIST` below anyway, do you need to check for it here?


Could you use a `NamedTemporaryFile` to avoid the try...finally block? Something like
```python
with NamedTemporaryFile(prefix=outDir, delete=False) as n:
    yield n.name
    os.rename(n.name, name)
```
should take care of closing the fd for you when `n` drops out of scope, I think.

When writing the above, I realise that neither version will clean up the temporary file if an exception is thrown during the yield. I'm not sure if that's useful debugging output or garbage that we should clean up.


@akbargumbira :+1: for Pillow!


Maybe nicer to move the lines which don't actually use `temp` outside the context manager?


I am not sure it is a project object's responsibility to decide whether an execution is manifest merging.


My program instructions


Yeah, thanks for digging that up.


@hayleeliu  如果已经校对完成请更新一下wiki


how its work


Linking to the issues seemed of little value overall. I'm not sure what that would provide.


Nicely done.  I'll merge this and update the .pot file, along with the translations at:
http://translate.twinpictures.de/projects/colomat


Maybe we could split the condition in two.


Enemy must extends character


this is not tested yet, cause the plugin triggered an exception, so I can't promise that it works


Add 'protected' to every field and create gettets - that's your task for today and tomorrow morning


@dhensby Well, I’m pretty sure we need the `isset($_SERVER['HTTP_HOST'])`, I don’t really know what we should do in the event it’s not present though. How big a security risk does this pose? Should we need to block all requests without an `HTTP_HOST` set?

Just above this code block we try to extract a URL from `$_FILE_TO_URL_MAPPING`, if that’s not set, but `SS_ALLOWED_HOSTS` is defined, then all CLI requests will fail with a 400 error I think? Does `sake` _require_ `$_FILE_TO_URL_MAPPING`?

Ping @nyeholt @tractorcow, I feel like you guys may have the answers here :)


@Oksisane I reimplemented the MonkeyDo class so that it gets called directly from the MonkeyC instance.  I can't do this with MonkeyWrite because then we would be rewriting the entire JSON when we have a new instruction.  @Oksisane Is there a way we can append to the JSON immediately when adding an instruction without rewriting any parts of it?  This is so if the robot communication stops, we already have the instructions stored.


Why did you get rid of the password? Isn't one needed?


I check some other features. In exadel-related features I see the same text. Should we ask Richard Fontana about it?


Hi @kixam

usually plugins in tiddlywiki only have 3 version digits major.minor.patch. The tw version parser will not allow import 0.7.3.1 over 0.7.3 as he regards them as equal.

-Felix


Actually i think the first two conditions are not needed at all. Our configurator will be only asked to create a build participant for the executions mapped to it. And we only map to the AMP in `lifecycle-mapping-metadata.xml
`. I think we should also remove the last condition, since people may bound the manifest merging to another lifecycle phase, like `process-sources`. This will leave only one condition, but that is really needed.


，1


My local set up for MySQL has a root account, but there is no password for it.  That is why I removed the password.


The code you tested out here is really advanced, and definitely not what you need to get this app working.

Let's spend some 1:1 time (either during your scheduled time or during my open appointment time -- let me know) talking about this. Based on how you've performed in class I know you're familiar enough with HTML/CSS/JS basics to get started on this assignment.

If you can come prepared with some specific questions that would be very helpful!


< table class = "tablecomment">


TeamCity Jarvis :: Jarvis.DocumentStore :: Debug / Test / Release [Build 572](http://build.prxm.it:8811/viewLog.html?buildId=3404&buildTypeId=Jarvis_DocumentStore_CI) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 289 Build time: 00:13:41
### Failed tests
```
Jarvis.DocumentStore.Tests.dll: Jarvis.DocumentStore.Tests.SelfHostIntegratonTests.DocumentControllerIntegrationTests("v2").verify_de_duplication__not_delete_original_blob_before_15_days: <no details avaliable>

```


this is docker specific, right? I guess that should somehow belong to the DockerSource class


> sure you have ruby installed and the do

sure you have **R**uby installed and the**n** do


I just saw that... I did this because 0.7.3 was working in "every" mentioned case, but actually not "every every" case, but there is so little difference that I wanted to add a minor patch version number...

do you think I should release a 0.7.4 ASAP ? I had hopes nobody would have downloaded 0.7.3 within its 1 hour lifespan... and there is still the possibility to delete plugin > save > install plugin...


fair enough


hi


This would be the case if the monitor for the LRP was a healthcheck on the port that the application is listening on. However, the default LRP that gets created for inigo has a monitor of `true` and thus the LRP becoming RUNNING instantly. We are going to investigate whether or not it makes more sense to add the health check to the LRP or leave the test the way it is.


added to seed


couldn't we just redirect to main page if no GET variable is set?


Is this ``halign`` specification added intentionally?


@jeremyfelt: I'd like to also add the option to not use an H# header, but to have the header wrapped only in `<header>`. This would enable us to use it as a item header without having to compete with any header styles and without having to be opinionated about where exactly the header falls in the document hierarchy. This would be a bit like using `<header>` in an `<aside>` to indicate that it is the header of that item without having the aside's header competing with the document hierarchy.


That probably is a better idea, I'm not sure if it will affect users flow if they attempt to go to a page and are bounced back to the main


@elpaso actually Pillow is used in the plugin and users apps. Btw, I've got problems spinning it up using the current requirements. In doing migration I got these problems:
* django.core.exceptions.ImproperlyConfigured: Error importing debug panel debug_toolbar.panels.templates: "cannot import name get_standard_processors"
Upgrading django-debug-toolbar solved that (https://github.com/django-debug-toolbar/django-debug-toolbar/issues/668) So I updated all the requirements in the venv.

But after that, still in doing migrations I got this:
* django.db.utils.ProgrammingError: relation "django_content_type" does not exist
Following http://stackoverflow.com/questions/26283159/django-1-7-migrations-how-do-i-clear-all-migrations-and-start-over-from-scrat and doing:
    -  find . -path *migrations* -name "*.py" -not -path "*__init__*" -exec rm {} \;
    - recreating db

 solved that

Doing migration is fine now. But now running the server throws this python error:
```
Request Method: 	GET
Request URL: 	http://localhost:8000/
Django Version: 	1.8.2
Exception Type: 	ProgrammingError
Exception Value:

relation "cache_table" does not exist
LINE 1: SELECT cache_key, value, expires FROM "cache_table" WHERE ca...
```
any hints on this? Thanks for the help.


This is commented out, is this something that is still necessary?


Victor, Ok, I can certainly agree to that!  (My only point was that these settings are also needed by people using raster maps, who do otherwise not need any renderer at all.)
- I have also verified that these settings "GPX track color" and "GPX track width" affect :_every_ track, not just the currently recorded track, which I think is very good!  (So I guess we should rename the string contents ?)
- Leonid, I assume we could simply copy the rendering attribute code to default.render, and all dependent map styles would participate without any changing them all? (renderers like Touring view define/overwrite with their own default track color, but northing else).

Let me know if you need my help implementing anything. Thx - Hardy


Is that a fix for what you were talking about or do you want me to have a look at it?


## This is a markdown file


tava dando o erro aqui?


Ah well sure, the reason behind the change in PHPUnit is actually a good one so I'm not really complaining about the (mostly quite obscure) BC, I've somehow taken into account that I'd eventually run into potential breakages when I decided to start overriding `protected` stuff. The previous code in `PredisTestCase` was crappy to begin with and, in the end, it wasn't such a bad thing that I was forced to change it ;-)


This is not tested yet, cause the plugin triggered an exception


Commits like this should really **only** add documentation and not also change variable names and/or other stuff! So please be sure to split up unrelated stuff into a separate commit next time, thanks. Cc: @domenkozar


why is the mapRectToScene used inside this function? this will break the possibility to use a web view inside a qml item.


Thanks for the help @lancelotsix

I looked at some other applications that are using bundlerEnv.

```bash
$ grep -r "bundlerEnv {" *
doc/language-support.xml:bundlerEnv {
pkgs/applications/networking/cluster/panamax/api/default.nix:  env = bundlerEnv {
pkgs/applications/networking/cluster/panamax/ui/default.nix:  env = bundlerEnv {
pkgs/applications/networking/mailreaders/sup/default.nix:bundlerEnv {
pkgs/applications/version-management/gitlab/default.nix:  env = bundlerEnv {
pkgs/applications/misc/jekyll/default.nix:bundlerEnv {
pkgs/tools/security/metasploit/default.nix:bundlerEnv {
pkgs/tools/text/gist/default.nix:in bundlerEnv {
pkgs/tools/misc/t/default.nix:bundlerEnv {
pkgs/tools/misc/tmuxinator/default.nix:bundlerEnv {
pkgs/tools/misc/riemann-tools/default.nix:bundlerEnv {
pkgs/tools/misc/lolcat/default.nix:bundlerEnv {
pkgs/tools/misc/fluentd/default.nix:bundlerEnv {
pkgs/tools/audio/mpdcron/default.nix:  gemEnv = bundlerEnv {
pkgs/development/interpreters/ruby/bundix/default.nix:bundlerEnv {
pkgs/development/tools/chefdk/default.nix:bundlerEnv {
pkgs/servers/consul/ui.nix:  gems = bundlerEnv {
pkgs/servers/monitoring/sensu/default.nix:  bundlerEnv {
pkgs/servers/monitoring/riemann-dash/default.nix:bundlerEnv {
```

Based on what I found I think we need to change `builderDefsPackages` to `callPackage`.

```bash
diff --git a/pkgs/top-level/all-packages.nix b/pkgs/top-level/all-packages.nix
index 7dc78d7..ddea927 100644
--- a/pkgs/top-level/all-packages.nix
+++ b/pkgs/top-level/all-packages.nix
@@ -2212,9 +2212,7 @@ let

   mscgen = callPackage ../tools/graphics/mscgen { };

-  msf = builderDefsPackage (import ../tools/security/metasploit/default.nix) {
-    inherit ruby bundlerEnv makeWrapper;
-  };
+  msf = callPackage ../tools/security/metasploit/default.nix { };
 ```

I've looked at `*/fluentd/default.nix` and `/sensu/default.nix` and think the `metasploit/default.nix` should look something like this:

```
{ stdenv, lib, bundlerEnv, ruby }:

bundlerEnv {
  name = "metasploit-framework";

  inherit ruby;
  gemfile = ./Gemfile;
  lockfile = ./Gemfile.lock;
  gemset = ./gemset.nix;

  meta = with lib; {
    description = "Metasploit Framework - a collection of exploits";
    homepage = "https://www.metasploit.com/";
    license = stdenv.lib.licenses.bsd3;
    maintainers = with maintainers; [ rdesfo ];
    platforms = platforms.all;
  };
}
```

I was able to build sensu by running `nix-build  -A pkgs.sensu`, but I still get the following when I try to build metasploit:

```bash
$ nix-build -A pkgs.msf
these derivations will be built:
  /nix/store/k4rnpqxff6qncvl8kzmb73b47izc42aq-metasploit-framework.drv
building path(s) ‘/nix/store/3yknzgkjbb3ab7w0q5zgvazk4vbrwdrh-metasploit-framework-bundle’, ‘/nix/store/dd6yz38mwlklxgh1ncp1gwz3914bx7dn-metasploit-framework’
installing
running pre-install script for ffi
There are no gemspecs at
/nix/store/3yknzgkjbb3ab7w0q5zgvazk4vbrwdrh-metasploit-framework-bundle.
builder for ‘/nix/store/k4rnpqxff6qncvl8kzmb73b47izc42aq-metasploit-framework.drv’ failed with exit code 15
error: build of ‘/nix/store/k4rnpqxff6qncvl8kzmb73b47izc42aq-metasploit-framework.drv’ failed
```


> Having bigger value may not necessarily be an advantage.

Indeed. We don't want a too big number to limit the size of the `IN` query, but we want it large enough to limit the number of queries to the DB.

> [...] to understand how much each thread really takes when we throw some load

On that note, I think we have a lot of metrics but they are either redundant or confusing. Maybe we could simplify them (and add others if need be)?


The intent is that we'd like to keep the `sudo`-allowed executables to a limited set, and explicit.

Ideally we'd have some other way to deal with this other than `sudo`.


Provavelmente sim, é na hora de deploya o glassfish.


@aszlig did something break?


ACK.


Thank you, I've closed the issue!


@domenkozar: See b028c14256d08c1dcb5a26f37edc879885cac20d.


I see, thanks!


Maybe `--output-encoding` -> `--input-encoding`.
What does happen when only output encoding is specified and input encoding is not specified?


@unintendedbear Sí. Si puedes volver a repetirlas para dejarlo claro, mejor.
MUSES centralizes the access... stores the delicate files in ... Also, it is based in a cloud environment (the server), and blahblah (no sé mucho de MUSES, como puede verse).

También coméntalo en la respuesta a los revisores.


Since you are pulling from the db on the 'user' field, user should probably be declared as the primary key, else duplicate names will break this code.


`func.name` is `undefined` in IE.


Ah I see, I forgot to create cache table. It's up running fine now!


@stevenpetryk Ah, yes, I am sorry. This must have fallen out in the mess.. I have fixed the jar on Clojars.

Anyways, even without that you could get the result with:

    (Minim. (DefaultFileSystemHandler.))

it's just the static convinience method that was missing... but now it is there.


@littleguy77 I was going to open a pull request for this but I was wondering if there is any reason to make this hack not android only?


@Kronuz I'm having trouble understanding what exactly your patch is supposed to fix. As far as I understand, in Python it is illegal to use a relative import to refer to modules located in a different directory on the filesystem. (Moreover, `import .some` is also an illegal statement.) Therefore, your attempt to import `modules.first.some` via `from .some import ...` in a file not located in `modules/first` is invalid Python.

Please provide an example of what you are trying to do that was not working, but that your patch fixes. (Description of file system tree, Python files with contents, etc.) I need this information in order to write test cases in order to incorporate your fix.

Thank you.


There is indentation issues in templates that resulted in this output.


update


@dmalikov, do you have any idea why setting the `buildTools` attribute does not help? It seems to work for my dotfiles, but not here.


Right


Is the function name + documentation still valid after this change?


Per slack, gonna change the defaults and roll new packages, then test the installer and get it merged


Looks like the commit message should make mention of this block; it's what actually displays the extra field values on the admin page.


I would like to wait, till we have the class in a good shape. For clients there is a binding which can be changed. In other words, there is nothing to change for simple Xtext languages, at least not yet.


That sounds reasonable, then we don't need the `getPlugin()` method.


typo: a simple way


thanks :)


>  but there is so little difference that I wanted to add a minor patch version number...

hehe, yes, I understand. Its fine, also I did not want to be nitpicky. Your intention is understandable.

I once had a long discussion with Jeremy exactly about this. My argument was that tiddlywiki should also consider the build incremental of the version string (like debian packages have) when comparing plugin versions (e.g. 0.1.3+12 > 0.1.3+11) so patches of patches (like in your case) do not need a version incrementation. But this was not accepted. However, we agreed on that a version containing the addition "prerelease" always wins (e.g. 0.1.3-prerelease > 0.1.3).

> do you think I should release a 0.7.4 ASAP ? I had hopes nobody would have downloaded 0.7.3 within its 1 hour lifespan... and there is still the possibility to delete plugin > save > install plugin...

Yap, no problem.


fixed


Should we use booleans or ints instead of strings here?  This implementation seems clunky


Yeah! I will push the change right now.


why?


Do we know why we were using `button` in here before rather than `li`?


TeamCity Bender :: Deploy [Build 3.0.11.0](http://build.mikeobrien.net/viewLog.html?buildId=2433&buildTypeId=bender&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 3237, ignored: 5 Build time: 00:01:23


Should we set variables here and them test them against their expected value?


Dropping the table seems pretty risky here-can we include maketable.php and then query the db for a test post?


I think we should remove the block above, given the discussion we had about making the concept simple to understand (the use of the decorator should override the behavior on any exception).

Well, to be honest I'm still not sure that should be the way to go.


@nyanpasu right? :D


I believe this comment is slightly misleading as the listeners should be attached to the _child_ of the `md-fab-action-item` because the child `<button>`s are the things that actually gain focus. I believe the code is correct, just the comment is odd.


The choices in argparser are enforced strictly, so there is no chance of case sensitivity issues


I’m pretty sure I copied it from some very old documentation back when I was writing this template- and it worked (except for FF, apparently).


spelling: parameter
(Mater is from Cars) :smile:


I exploded a creative server this way, you can't rejoin, even the owner xD.


@westonruter No dice. I guess we will have to resort to using an inline comment wrap for now until we can properly sort this out. Thanks for the idea.

```php
// @codingStandardsIgnoreStart
// the code...
// @codingStandardsIgnoreEnd
```


Right, I did that, oops. Hmm.. should they be listed as choices like that?


Perfect, thanks!  I think you should submit this as-is.  Much easier for upstream to remove the ifdefs later than to figure out where to add them.


see next line ` child = angular.element(child).children()[0];`


Why do you need to check whether or not `args.buildTools` exist? Mostly because of the fact that it doesn't exist all the time.

I have same issues with `overridesPackages` function here https://github.com/dmalikov/dotfiles/blob/master/shell.nix#L4

Are you sure that `args.buildTools` is not empty for your dotfiles?


I thought this would only work with a tuple (never checked actually)


Yeah, that’s probably the best way, though subcommands might be better (but I don’t know whether you can do subcommands within subcommands :P)

From:  sarahmarshy
Reply-To:  0xc0170/project_generator
Date:  Tuesday, 7 July 2015 17:50
To:  0xc0170/project_generator
Cc:  Matthew Else
Subject:  Re: [project_generator] Add no unicode option for list and auto no unicode detection (e54a738)

Right, I did that, oops. Hmm.. should they be listed as choices like that?

—
Reply to this email directly or view it on GitHub.


This isn't production code, but it is gonna be much better for PHP scripts to out put data JSON encoded when used in the future.


this table is only created and use for testing purposes.  we don't want multiple copies of the table or the values in it fool


че это за говно?
`padding: 0;` зачем 2 нуля?


и про margin то же


I expect
```nix
(if args ? buildTools then args.buildTools else []) ++ [ nixpkgs.pkgs.git ]
```
to evaluate to `[ nixpkgs.pkgs.git ]` if `buildTools` is missing. Is that wrong?


this comment is unclear


Should we test against the live db instead?


Pq vc num quis implementar e usou uma gem?


This commit is identical to d40b5b568a2084140e554a85591dacb881ca207c and per [this comment](https://github.com/mysociety/FixMyStreet-Commercial/issues/671#issuecomment-83044735), should be part of `issues/commercial/671-response-templates-redux` (#1129).


второй раз стили?


> Is this halign specification added intentionally?

No, I probably caught something while scrolling. Thanks for catching that.


It's a fully implemented and ready to use class. The only reason i've created a sub class in a source folder is to make customization possible.


De uma olhada na nova syntax de hash: https://github.com/bbatsov/ruby-style-guide#hash-literals


куски комментов?


You are correct it seems I missed a commit.


does this break our current examples? published docs?


DesignDaysGridView (evanweaver) - i386-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


DesignDaysGridView (evanweaver) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


That's because it can't be cast to a string, try reading the readme: https://github.com/Quinny898/PersistentSearch


Done.


@tomsmeding 77430f359c13cce0aa31f33c3ddf0f66fc7846b2


Something like:

```python
try:
    if msg['data']['name'] in hooks._before_validation:
        ...
except KeyError:
    pass
```
or better,

```python
try:
    name = msg['data']['name']
except KeyError:
    pass
else
    if name in hooks._before_validation:
        ...
```

...would be probably closest to idiomatic Python. In Python it's very normal to [often go with forgiveness then to ask permissions in advance](http://programmers.stackexchange.com/q/175655/150576).

Maybe also storing `msg['data']` in advance would work nicely:

```python
data = msg['data']  # if not sure it's present, I'd use try/except with KeyError here

if data.get('name') in hooks._before_validation:
    ...

...

if data.get('name') in hooks._before:
    ...
```


`global.setupAttachmentEditor = function setupAttachmentEditor() {`

I think for "real" functions having a name is nice


hacky... but this works. I was trying to solve this a different way yesterday.


@ksooo no problem
@ryangribble I'll create a separate PR that uses unique_ptr instead


thanks for the input.

I do not think "prerelease" is appropriate for this situation, but I might use it in the future now that I am aware of it :)


O Minitest usa `should`? O rspec recomenda não usar should nas descrições dos testes e sempre ser imperativo: http://betterspecs.org/br/#should


@9prady9 Do not do that. We need to fix this upstream in C++ API. For now link with a specific backend at compile time.


`editAttachments`? Even though this thing will eventually be replaced anyway..


in addition to `export * from './annotations'` below?


Good caught, it actually has to be a tuple. I'll change the func detection to tuple too.


Não tem um jeito melhor de fazer isso?? :fearful:


Hmm, fixing in the C++API will not solve RUST. So ignore the old statement.


Oh, I think I misparsed your question. I check its existence simply because it fails the evaluation otherwise.


Wow. There were a whole lot of silly mistakes in here! Thanks!


This commit actually breaks zfcampus/zf-mvc-auth version 1.0.3, which relies on the fact that IF the service manager has something configured for the key 'ZF\OAuth2\Service\OAuth2Server', it is an instance of \Oauth2\Server.


I believe everyone agreed with this. It's documented and it's what most of us consider as expected behavior.


oh, no more Mater


```python
obj.setdefault(name, [])
```


use unless ao inves do if! se vc tem apenas um caso.


nice docs


Use o I18n para mensagens :wink:


You forgot to change the second `str`.


Aki tbm


```
onClose: function(data) {
    if (data !== null) {
        location.reload();
    }
}
```

Otherwise you reload even if nothing changed. However, you'll probably need to tinker a bit with ajaxDialog so it passes some kind of data when you close it after modifying something... not sure what's the best way to do that (I can have a look tomorrow if you want since I wrote the whole ajaxDialog thing)


Okay


does this do something? does dgeni exclude this from HTML and .d.ts?


why stdout and not stderr?


Having problems with the Yosemite bottle:
```
🐙  curl -IL https://homebrew.bintray.com/bottles/libgphoto2-2.5.8.yosemite.bottle.tar.gz
HTTP/1.1 401 Unauthorized
Server: nginx
Date: Tue, 07 Jul 2015 16:57:19 GMT
Content-Type: application/octet-stream
Content-Length: 0
Connection: keep-alive
WWW-Authenticate: Basic realm="Bintray"
```


DesignDaysGridView (evanweaver) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


Eita noises! Tá mt estranho isso aki


@SvenRtbg This is why the change was introduced for the 1.1 version (it's not on the 1.0.* series) of zf-oauth2, and messaged in the release notes.

If you were overriding the service previously, alter your factory to return a callback that will return the `OAuth2\Server` instance.


what does this do?


This should be removed


Can you merge this commit to `master`?


again, `transaction.setdefault(key, [])` (everywhere)


should be
```
[AC_CHECK_LIB([fuse], [fuse_main],
```
instead.


Using the official sqllocaldb API IMO would be the cleanest solution. If we can't include packages under Apache license, we could write our own .net wrapper.


would probably make more sense to use Python microframework in the future, but good enough for now


+1 for extension tags, but why there are two of them?

By the way, I read that `@register.filter(name='expiry')` can be left just `@register.filter`, then method's name will be used as a name of a filter.


requirement of the dredd interface ... I should remove the `file=sys.stdout` because it's not needed


ditto


Yup, this is different, i think it is in fact similar to the solution you are thinking for c++.


Is there a reason these couldn't be implemented as tags?  I think that strikes me as the simplest way to do it, especially since there are multiple potential classifiers for a single issue.


А зойчем `checkStorage()` разнесено в 100500 мест?

В самом начале перед `if(doc.readyState === 'interactive' || doc.readyState === 'complete') {` нельзя его поставить?


Did you try without shield?


I already have this commit locally, well not exactly this obviously, but similar (only one file changed):

commit 07c5cf6077211ea238771caffef3a4d7f03d658e
Author: Derek Parker <parkerderek86@gmail.com>
Date:   Mon Jul 6 14:19:22 2015 -0500

% git status
On branch master
Your branch and 'origin/master' have diverged,
and have 1 and 1 different commit each, respectively.
  (use "git pull" to merge the remote branch into yours)
nothing to commit, working directory clean
{atlas} [~/src/github.com/derekparker/delve]% git pull --ff-only
fatal: Not possible to fast-forward, aborting.

It seems that this upstream commit has been deleted and replaced with another one, which now prohibits further regular updates before this divergence is resolved. I know how to fix this, but it involves additional commands which shouldn't be required. Anyway, editing already published commits is considered not polite.

From https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History

"You need to be careful with this technique (git commit --amend) because amending changes the SHA-1 of the commit. It’s like a very small rebase – don’t amend your last commit if you’ve already pushed it."


maybe the example could be loaded to the test from this file so it's not pasted directly to `.feature`, but I don't know what's best practice for that - if it was me, I'd leave it in separate file as an included fixture


Можно. Просто сделал его вызов по необходимости, чтобы не запускать его на каждой странице в браузере.


Shouldn't this move to ``terminate`` as well?


А.


rip


this doesn't work if you have multiple sliders on the same page. When you update an input all the sliders are updated as well. I reverted to `slider.foundation("slider", "set_value", $(this).val());` but I changed the code to
```
$(this).off('change').on('change', function () {
  slider.foundation("slider", "set_value", $(this).val());
});
```
to avoid accumulating change triggers


Not sure if it is worth mentioning that intersects is equivalent to not disjoint.


Move to separate commit.


This comes from the dredd integration test, so I'm not sure about it, it would imply changing the tests and maintaining two versions of it.


Working nicely.


That's surely possible. It's open source, you can add it yourself..


This change looks good, though the branch-on-branch instead of PR makes things confusing! My github-foo is poor.


Okidoc ;-)


The small container name was reading OSE-3000


Yeah, maybe we will come up with better solution, but now it has to be in the `.feature` file.


+2. (Mental note not to forget this case anymore)


Yeah, absolutely my bad there. Apologies.
On Tue, Jul 7, 2015 at 11:06 AM Zlatko Čalušić <notifications@github.com>
wrote:

> I already have this commit locally, well not exactly this obviously, but
> similar (only one file changed):
>
> commit 07c5cf6
> <https://github.com/derekparker/delve/commit/07c5cf6077211ea238771caffef3a4d7f03d658e>
> Author: Derek Parker parkerderek86@gmail.com
> Date: Mon Jul 6 14:19:22 2015 -0500
>
> % git status
> On branch master
> Your branch and 'origin/master' have diverged,
> and have 1 and 1 different commit each, respectively.
> (use "git pull" to merge the remote branch into yours)
> nothing to commit, working directory clean
> {atlas} [~/src/github.com/derekparker/delve]% git pull --ff-only
> fatal: Not possible to fast-forward, aborting.
>
> It seems that this upstream commit has been deleted and replaced with
> another one, which now prohibits further regular updates before this
> divergence is resolved. I know how to fix this, but it involves additional
> commands which shouldn't be required. Anyway, editing already published
> commits is considered not polite.
>
> From https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History
>
> "You need to be careful with this technique (git commit --amend) because
> amending changes the SHA-1 of the commit. It’s like a very small rebase –
> don’t amend your last commit if you’ve already pushed it."
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/derekparker/delve/commit/05517c62c13bd5da8aafd4a7fc9e9bea5d710074#commitcomment-12043993>
> .
>


Hi Martin,

I changed the italian flag and the french one as well because the blue stripe was smaller then the red one.

Best Gregor


А вообще надо бы составить список доменов всех поддерживаемых борд и убрать этот `// @include         *`. Уж очень мне он не нравится.

Пользователи обезьян пусть добавляют мелкоборды сами (либо реквестируют в треде), а для хромокостыля можно сделать кнопачку с возможностью добавления страницы.


The main reason I chose to automate the dedi server, is because it is a hard dependency for the node app. The node app can't function fully without the pcars server running, so it just makes sense to me to have the node app control the dedi server just like any other dependency.

It also opens the possibilities of programmatically starting/restarting/updating the server software from a web interface - I see this useful for updating the blacklist and whitelist, as well as configuring it, like how it currently is with grunt.

Screen isn't really necessary at all. In a deployment situation, I would use something like [pm2](https://github.com/Unitech/pm2), or register the node app with [upstart](https://help.ubuntu.com/community/UbuntuBootupHowto) (/etc/init.d) which will run the app as a background process and other things like that.

The main issue I have with the VM solution, is that it requires a bit of configuration. I think I can automate that, or even just bundle a pre-configured image with the repo. Basically I want the make starting the server from a local windows machine quick and painless, with little to no configuration. I think vagrant is a good way to do that.

So, moving forward, I would like to make the pcars server run as a child process of the node application, so that if the node application is closed, the dedi server is closed with it. Currently, the dedi server is run as a background process in parallel with the node app, which is fine, but will stay open if the node app is closed.


I'm fine with it then. In fact, it's my preferred choice.


why?


for the same reasons getDisplayName is abstract. The implementation doesn't make sense and as far as I saw it gets always implemented by the generated sub class, no?


better tracebacks for example


i could go with either approach.. my only nit pick on the current form `toggle` is it expects an input parameter.. to me `toggle` is flip current state (whatever it is) of a binary entity... and therefore should not require an input paramter


Strictly speaking shouldn't this be `("Windows" , )` otherwise you're searching a string for a substring.


That's what I thought. This should be replaced with a proper dropdown. I didn't do it because it's a mess to translate the whole logic.


I would appreciate feedback on whether or not this will work.


Черт. Этого я больше всего не хотел. Этим мы *существенно* ограничим скрипт по аудитории.
Мелкоборды постоянно меняются. А зарубежные так вообще.
Весь скриптотред будет состоять из сплошных просьб и нытья.


Ok for now. But than it should be nice Java.


Hey @mohsen1, i don't quite get it, what it the indentation issues? Each of them is 2 spaces for a level, no?


1712-AvailabilityManagers (kbenne) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


Need to specify that the default is None


"they satisfies" should be "they satisfy"


Why not argv[0] on these ./blah entries on the help string?


Other than that, :+1:


whoops... That's what I meant when I said this probably still needs testing ;)


We usually use 'default' for generic implementations. And for the pattern "src_gen gets extended by src_once" we use "Foo extends Abstract<Foo>".


When one supplies Exception instead of tuple, this won't work


Modifié l'espace voyageur où l'echo PHP était mal fermé et ça affichait un tableau seulement pour la 1ère occurence


So if i read the post by @mk01 correctly we should:

__RPi2__
change
```
deb mirror://apt.xbian.org/mirror.txt      stable  main rpi-wheezy
```
```
deb http://mirrordirector.raspbian.org/raspbian/ wheezy main non-free
```
to
```
deb mirror://apt.xbian.org/mirror.txt      stable  main rpi2-jessie
```
```
deb http://ftp.us.debian.org/debian jessie main non-free
deb http://http.debian.net/debian jessie main non-free
```
__RPi 1__
change:
```
deb mirror://apt.xbian.org/mirror.txt      stable  main rpi-wheezy
```
```
deb http://mirrordirector.raspbian.org/raspbian/ wheezy main non-free
```
to
```
deb mirror://apt.xbian.org/mirror.txt      stable  main rpi-jessie
deb mirror://apt.xbian.org/mirror.txt      stable  main rpi-wheezy
```
```
deb http://mirrordirector.raspbian.org/raspbian/ wheezy main non-free
deb http://ftp.us.debian.org/debian jessie main non-free
deb http://http.debian.net/debian jessie main non-free
```

So,
- Fully change the `RPi 2` from wheezy to jessie.
- Add jessie to `RPi1` sources and thereby leaving the wheezy sources in as well.

I can update the `xbian-package-repo` myself to install the new repositories if correct.


1712-AvailabilityManagers (kbenne) - i386-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


let's get back to the `callable` version


wouldn't this break when deleting the attachment?


Used bundle install


А старые версии скрипта будут недоступны для вновь открывшихся борд.
Желающих править ручками где бы то ни было всегда очень мало.


Looks like according to the NTP protocol there's a flag that gets sent
during the month of the leap second addition/deletion. The NTP server then
stops time for that extra second before incrementing normally. I hope I'm
understanding this correctly, anyway.

On Tue, Jul 7, 2015 at 8:08 AM, Dan Quigley <notifications@github.com>
wrote:

> To be honest, I'm confused about how leap seconds are manged wrt NTP
> myself. I found this https://www.eecis.udel.edu/~mills/leap.html that
> holds some promise to provide the answers I seek, but I have not digested
> it completely. What bothers me is that without any leap second processing
> in my implementation, the NTP server seems to provide what the Windows NTP
> client expects.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/tmiw/arduino-ntpd/commit/0940a8a5d50cd9ce021760ab9da4f6079de4ad45#commitcomment-12041595>
> .
>


We can go over this on our call this week.


Тогда ладно.


1712-AvailabilityManagers (kbenne) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-NA-red.svg)


97221720_32BitInstaller (macumber) - x86_64-MacOS-10.9-clang: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-warnings-yellow.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-0.0%25-red.svg)


:police_car: :cop: :hand:


@hyi Assuming that the file size returned by ils -l is the actual file size (and not the on-disk size), this particular change looks okay.  More broadly, I think the download method should check to see if path exists before assuming that it does.  Also, there should not be any HTML markup in the response generated on line 29.


I would use Params::Validate's validate_pos here unless the error messages it provides are insufficient.


Reststop 1.12 has been release, and is available on central.


fixed.


Look at this:

https://github.com/pbs/pycaption/commit/fc0f8183d7e46dc8e0f3fb6259ffbe6d310d8158

Normally the clients should already have problems with bs4.
So I don't understand why it is coming up with pycaption now.

But maybe it is better solution to disable pycaption if Python 2.6 is used.
Or other option... We don't support Python 2.6 clients.


should not split `-Xcompiler -Xlinker -Xcompiler '--exclude-libs=libcommon.a'`:
the `'--exclude-libs=libcommon.a'` option following `-Xcompiler` is passed to compiler.


```
    :param exceptions: Tuple or single exception to be assumed as
                       test fail [Exception]
```


:+1: :smiley:


GraphicsMagick (version 1.3.21 2015-02-28 Q8) would only rotate the first frame of an animated gif in my testing.


Yeah, was unsure if Mick wanted to stick with the $& method or use JSON, so I just did both.  If the JSON works out, we can remove in future refactoring.


It's just crying wolf. All is fine.


Makes the decorated test to FAIL upon raising certain exceptions.

The default avocado behavior is to ERROR upon any exception that is
not an avocado TestBaseException, so this decorator provides users
with the means to have less stringent requirements (Example, we want
to turn SSH related exceptions into FAIL, when it would otherwise be an
ERROR).


This updates more than just Id, so slightly misleading name...


I personally found these lines a little confusing.
What if arcanist setup instructions change? Is there any value in having them here since they are all documented in the url?


This is helpful to check that the conda *binaries* are working, but is not so helpful to run on every pull request and source code commit. We'd need to just install the dependencies via conda, then build RMG from source. But I guess it's a helpful start if you can get it working...


True enough, it was just Session ID but it made sense to combine them all!  Will change to updateSessionInfo...


I think there's an extra "None" in here?


Did it work?


Testing...


missed this in the documentation. Input to signing data should be converted to ASCII characters.


@coditect That's odd. applyDataFunc() ends up calling apply_image_func in bridge.c, which iterates over all the images, coalescing if needed. I'll run a few tests on my end to make sure it's working properly.


`addNotificationTarget`.


I imagine the first check should be are we cli, then we check the host.

The cli does throw errors if mapping isn't set, but ultimately this protection is not required for cli requests


Is this actually needed? It seems like Python's usual way of handling missing attrs would work.


Will it be activated automatically when no setting is defined and the pcre lib supports it?


just draft


ACK


It'd be better to also pass the container to this method, since calling content.one is expensive. That way you'll only need to call it once, instead of for each field node.


Similarly, is this actually needed?


"Point" is definitely overloaded around here, but as one of the underlying concepts is "point set theory", it's not worth purging not-vertex instances of point from the docs IMO.


LGTM other than one comment.


So noted.


good job!


The original version in theory could have left the process without valid stderr. Now the fixed version breaks stdout too if the theoretical problem ever gets triggered.

Since the daemonize() function is full of other issues and it is needed for sysv init support only -> might be better just to remove the whole function and associated command line option.


I have a serious doubt in this. The previous ```MessageVerifier``` accepts ```nil``` value in its ```generate``` function's argument. Now, It also generates a message corresponding to this ```nil``` value. But when we verify it, it raises ```ActiveSupport::MessageVerifier::InvalidSignature```. Do we want the same with our new ```MessageVerifier```. As of now, it is implemented in the same way as shown in this test.

According to me, a ```nil``` value should not be accepted in the first place. What do you think ?