Well, as soon as I get time to get hands on it, I think that it will be useful for a lot of people around the world, it's a pitty to use naga w/o the 12 buttons on the side.


*Command failed*: `composer install`
```
Array
```


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16641](https://teamcity.mesosphere.io/viewLog.html?buildId=141054&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:04:18


Sure thing! I'll do it now.


Or, even better would be to override the `__json__` of Result ! This would be simple, just call the parent's `__json__` and then overwrite the `file` with `json_data.file = os.path.relpath(json_data.file)`

Cleaner and better IMO


If I understand this correctly, does it mean that the button feature is still there, and this is just a timeout? As in "If you don't press the button I will, eventually".

It's important that we keep the button feature as it adds some level of "interaction" for the user to test the uVisor features :) Maybe we may rephrase the comment to say that this is a fallback handler that will press the button for us eventually: It makes sense to have it even when a button is present.


LGTM. We should probably carry over the "concepts" template from Carina as well.


@pynicolas Could you explain (and mb document) this method semantic?


Changes to this file could have bin in a different commit


Maybe `This convertFrom(RasterRgba open(filename))`


Remove sortCode.


a valid account number


Position x, y and z not need Math.round because function useItem is return Position rounded
Sorry for my bad english !!


Move regexp to top?


First readme update


do you really need this comment?


Why do we allow unknown? How is this ever a valid thing to do?


@lourinaldi go the forum ask there. This is not a support section


Changed in the HEAD version.


and this... and others


Yup, sorry, I wanted to leave that part unchanged, but at some point I was too eager to do `git commit -am`!


Sorry, just realized that's actually what it's written :) It's up to you if you want to make it even more explicit, but otherwise this commit LGTM.


On Mon, Feb 22, 2016 at 01:14:14AM -0800, Jan Pokorný wrote:
> `gpl` must have been typo for `plumbgpl`, then...

Probably.

> Anyway, that check is completely useless even in the fixed form, AFAICT.

Yes, I think that we can remove it.


If you prefer you can always switch back to it, I don't care!


I don't know if the image loader allows it and I already have a red-blue conversion.


So, working?


remove


Named imports last.


Okay, just checked. Its GOING INTO **_and_** LEAVING a Vehicle

Soo.. its the button for mcpe


@vilchik-elena Mainly because you can add "elements". Subclasses have other mutable fields.


If the image loader can get rgb directly then I won't need this.


pourquoi utilise tu le _.assign({}, body...) si tu veux faire un clone tu peux utiliser
_.clone() ou _.cloneDeep() pour cloner si il y a des objet dedans.

Dans ce cas la tu peux surement jsute faire un data.agentId = ...


Does this mean that we have a delay of `1 ± 100 ms`?


No, this has nothing to do with the actual mod itself. It just changes they way our build server compiles the mod and uploads it to CurseForge. This specific update was so we could target MC 1.8.9


they are already there


А, ясно. Я через Github только отслеживаю.


Is 10 seconds too short? Should we do 30 seconds? I'm not sure how long we want to give a human to press the button on their own before it automatically gets pressed for them.


printStackTrace() is not best way to obtain exceptions


You have a function that creates RGBA image from file, why not use that instead of duplicating almost the same code here.


Since kdesk is doing more work than hsetroot, I wonder if  there any other resources which could get stuck in the X server if kdesk quits, other than the wallpaper? Although since kdesk is not supposed to quit I guess this is an academic point.


remove commented lines


is this sufficient? do you care about the LE or the BE variant of ppc64? I am pretty sure you care about the LE-variant, no? if so, this should really look for "ppc64" only when built on BE, and otherwise for "ppc64". Also see the libseccomp definitions.


As we know the relationship wouldn't it be better to assert for "foo", then "foo.42"?


similar here...


Named imports last.


No problem, I think that Claire at least was having problem to compile it. They are too many different LaTeX installs to handle them all. I think LaTeX lack a PiP-style package manager. TeXlive is a good thing, but not that much: individual package updates are hard, adding packages from outside is not easy. Maybe one day there will be (or perhaps that does already exists) a CTAN based package manager!

Anyway, that’s done, master branch only relies on pdflatex, xetex one on xelatex.


Flag: IW{Pr1m3s_4r3_!mp0rt4nt}


Why is remove idempotent?


不過這樣每次遷移時之前新增的資料都會被重置，所以應該要把資料放在viewCotroller裡再讓fighterTVC.fightersArray代入vcFightersArray的參照，`fighterTVC.fightersArray = self.vcFightersArray // self.vcFightersArray先做好初始化`。


LGTM


Another layer of conversion takes time in regression tests.


New route for stopping application : DELETE on /applications/{packageName}
The former one stays alive for the moment


Still confused by the distinction between Remove and Delete.


MifosX issue has been created - https://mifosforge.jira.com/browse/MIFOSX-2527


Any number below 20 I would say. 20+ would be a bit boring. I can imagine many users (or even us) just not wanting to press the button out of laziness and then staring at the board for 30 seconds :smile:
10/15 sounds reasonable.


The problem I had was correlated to missing fonts (like Adobe Garamond Pro) that I don't want to install, since it's not FOSS.


@vilchik-elena This method is used when we remove empty blocks. When an empty block is removed, all the blocks which link to it have to be updated: we have to replace one of their successors.


He is GhostBuster...ehm GhostHunter aka the Shark.
Nice you are still moving around :)


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16642](https://teamcity.mesosphere.io/viewLog.html?buildId=141058&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:08:40


*Command failed*: `composer install`
```
Array
```


How much time does it add ?


How you generate is up to you, but image doesn't need access to this secret.


The drive name is at least used correctly because of this line:
```path[1] := name[Length(prefix)+1]; # drive name
```


@pynicolas got it! Please add it as doc


祝贺你通过！


Yes indeed, we're attempting to fix the #2134 issue by creating a new class UIDropdownScrollable.


Does "All" matter here? By virtue of returning plurality, isn't the "All" implicit?


O(width * height) unless the compiler is smart enough to optimize dynamic memory.


That is our applications Dropbox OAuth client id. You can read about the oauth flow here: https://www.dropbox.com/developers/reference/oauth-guide


This is not related to CA, it is a boolean flag that if set tells cockpit to run without tls.


Fixed.


Je dois faire la mise en forme qu'avec des div ?


why did we do this?


Copy paste code and no typo fix : cient->client ;)


I have no idea, to be honest. It started to act up with the API making more and more problems.


Typo here.


Should callbackFunc() be able to abort the iteration on returning 'false'?


Can we destroy boats now? @thebigsmileXD


You refer to my project [SendCH-Telegram](https://github.com/TiagoDanin/SendCH-Telegram) ?-?


@alex-konovalov : You are right. That's a weird way to write things, but is correct :)


Yea, that was my thought as well. It's only for dev and it might be a good thing and force people to test locally with 1.10?


Typo here;


An error or a no-op?


If I'm not already blind tonight, then the `alpha` parameter got lost in this change.
Or did I miss something?


Hi minrk-

Thanks for the fix! Unfortunately, I'm kind of new to the notebook business, and I'm looking for this cell.less file so I can edit it myself, but I can't seem to find it.

I'm looking here:

...\Python34\Lib\site-packages\notebook\static\notebook

as suggested by the file path above. However, in this folder, there are only two folders, "css" and "js", no "less" folder. Inside of the "js" folder, there *is* a file called "cell.js" but it does not have the lines of code you mention here. Am I looking in the right place?

Thank you.
Zack


Thanks!  Much clearer for me :-)


10 seconds feels like a long time when staring. I'll leave it at 10 and add some more comments and a more verbose description in the commit message.


This causes the full source path to be shown as well, which seems wrong for just `--full-checksum`.


You appear to have inserted a trailing whitespace here. Could you remove it, please?


Yeah that's a good point, not sure of the breaking changes 1.10 will bring though.


There's another trailing whitespace here.


Yes, I know, too bad that https://github.com/georgd/EB-Garamond doesn’t have bold yet, even if it seems quite ready (https://github.com/octaviopardo/EBGaramond12). Because Regular EB Garamond is better than Adobe’s one in my POV, so eager to have the bold one.

But that mean you were compiling with xelatex (whereas some people have troubles using pdflatex). You can still use the xetex branch, just comment out \setmainfont[BoldFont={Adobe Garamond Pro Semibold},BoldItalicFont={Adobe Garamond Pro Semibold Italic}]{EB Garamond}.


cannot get ProviderIDs FOR all interfaces


Juan


@smarnach Yup, changed in 74cc6a9f13381888c0574a649c93d4c9a2e7a453.


This is a great idea. Never thought to include that before. One less thing to think about.


this math can be simlified:

``` scss

@mixin ratio-box($ratio: 1 / 1) {
  overflow: hidden;
  position: relative;

  &:before {
    content: '';
    display: block;
    height: 0;
    padding-top: (1 / $ratio) * 100%;
  }
}
```


Why not to have a small wrapper function so we hide the internal here ? Thinking about that mainly because of OutputJSONMemBufferCallback.


Is this comment still correct?


Ce fichier ne dérange pas celles et ceux qui utilisent Windows ou Linux directement, est utile à celui/celle qui utilise Docker et sera utile à la personne qui dans le futur voudra tester votre site sans passer trois jours à tout mettre en œuvre.


Gonna be painful to merge the updated protocol version but nothing else to say here than the few small comments.


Is it really useful to keep this check ?


Is this noticeable in our tests ?


Sorry, this is not the debug message I really meant, but [this](https://github.com/mikebrady/shairport-sync/blob/e525bbe266f55de3316ec782264aa9eeb8c8e2fc/audio_alsa.c#L406) and [this](https://github.com/mikebrady/shairport-sync/blob/e525bbe266f55de3316ec782264aa9eeb8c8e2fc/audio_alsa.c#L444).


What a Pushkin, what a son of a bitch!


This is not the right condition. I moved the connectors to the AppConfig.ready, but the error is still raised. If I delete this completely, everything works perfectly.


At what level is this query atomic? Once asserted can the set of pendingNames change underneath your feet?


It is not notable in percent because of the overhead of files.


Thanks for the patch.
Please use a commit message that describes what the patch does. "Update text.xml" is close to useless.


Yes I think, in order to be sure that the column is no more used.


also you mentioned for stylus version:
``` stylus
// Mixin: Ratio Box
// Set a specific width/height ratio. Useful on background images and iframes.

ratio-box($ratio = 1/1)
  $ratio = remove-unit($ratio)
  overflow: hidden
  position: relative

  &:before
    content: ''
    display: block
    height: 0
    padding-top: (1 / $ratio) * 100%
```


looks ok


Commit teste


@gabrielamatias "(including purchase ledgers, cash accounts and the fixed asset register)"


Updated.


I do enable EasyPrivacy/Peter Lowe's out of the box, because my view is to reduce privacy exposure<sup>[1]</sup> as much as possible. So to open the door to such domain as `exoclick.com` is the opposite of the spirit of the project here: reducing exposure to 3rd parties such as `exoclick.com` has priority over the mere goal of "preventing ads from being displayed", and in the current case both goals could be accomplished by blocking the specific 1st-party script on that site.

In my dreams, it would be nice to have a system in place which facilitate something which I could call "site watch", where one or more individual choose to contribute by watching one specific site for whenever they work around a blocker. Some kind of spreadsheet where volunteers pledge to watch such and such site, and report when a site works around a blocker.

[1] which I measure by the number of distinct 3rd-party domains touched while browsing, *especially* dubious 3rd-party domains (such as `exoclick.com`.


Note this change. Trivial, but I brought it in from upstream.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16643](https://teamcity.mesosphere.io/viewLog.html?buildId=141084&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 225 Build time: 00:04:00


I think I've just commented them out.


'partner' and 'join' cannot be used in the same sentence, either one.


@gabrielamatias weren't these in an ad-hoc bullet point before?


Hi do you have an example of el finder connecting to sql server from asp.net webform application


From now on I'll use this convention as well then. What was your reason for changing it?


Grates make great modern staircases...


du hast iwie nicht rebast sondern meine änderungen einfach überschrieben oder? :D


Why isn't this - if parentName == "" || NOT pendingNames.Contains(parentName)


nit: If


nit: remove this empty line.


@TiFu ah ja das ist so natürlich besser! :D


`div` en dernier recours car il existe plein de trucs plus _évoquateurs_ avant de devoir y recourir. E.g. [main, section, header, nav, footer, aside, article, etc.](https://www.w3.org/TR/html5/sections.html)

> _“Tables should not be used as layout aids. Historically, many Web authors have tables in HTML as a way to control their page layout making it difficult to extract tabular data from such documents. In particular, users of accessibility tools, like screen readers, are likely to find it very difficult to navigate pages with tables used for layout.“_
>
> https://www.w3.org/TR/html5/tabular-data.html#tabular-data


Sin


The new markdown formatting in the list of the ordered description of steps does not compile as a list.

We have not used RFC2119 language yet in any of the other test specifications, you can lowercase the MUST.


Need to make clear that the services stay around. That is requirement to make it clear to the community that Hawkular will continue development.

I would like to put special emphasis that `Hawkular Metrics` is unaffected by this. Should I write a separate blog post about?


nit: Reset


Same :)


Many thanks for this!


`LookUpAddress`


we need to create triggers for that role in scout else there will be no alerts


output to jsp - formated


@vilchik-elena I would prefer to separate the 2 operations:
1. Check one binary expression (possibly looking for "operands" in sub-expressions)
2. Check sub-expressions


I would cut `achieved alot`


thx


LGTM


実質的にはここしか変わっていません


Do ProviderIDs really have a parent-child relationship?


This comment should also explain that you only trigger layoutNode if the the layout size has changed and the child has children.


edit Vincent to nxt Friday


I'm not sure what is the purpose of this function. Seems to be too permissive - maybe it should thrown an exception if some element is not found? I guess it might be a good topic for a discussion in IRL.


Change to `child.children.length > 0`


Why have we changed our mind about this? I though the idea is that we should not warn when the user is explicit about doing a cartesian product. That is how it is defined in the CIP.


Instead of `strong`, this should be `copy`


LGTM :+1:


test


@pynicolas but then I will have several issues for one expression: `"a" + b +"c" + "d" + e`


We probably need to also change the .coffee name to reflect the new machine name


@aolieman What is the use case that it solves?

https://github.com/aolieman/django-simple-history/commit/dc4eb85a6c0734332e4ffb35277c574ced6df63d


I just want to make sure I understand things correctly: are the region-specific pressure coordinates and the regional variables themselves functions of the universal pressure coordinate ```PFULL_STR```?

One way you could avoid this dropping maneuver would be to store the regional variables as functions of the region-specific pressure coordinates (and have the region-specific coordinates be functions of themselves (not the universal ```PFULL_STR```)).  Then in passing the region-specific DataArray to your plotting routines all you would need to do is select the region from the Dataset and rename its coordinate to ```PFULL_STR```.  If things were structured this way, selecting from the Dataset would automatically take care of dropping the other coordinates automatically.

It might be slightly cleaner down here, but perhaps upstream (in initially saving the data) it could complicate things, so you'd have to make that judgement.


Both "statistics-enabled" and "enable-statistics" can be set in WF8 and WF9 and both of them set the same property. "enable-statistics" is only option since WF10. "enable-statistics" is in configuration files since WF8. This is the reason for this change.


typo? composit => composite


***Code Quality - Nitpick: ***
Remove redundant console.log messages


@ExSport sharks have to keep moving around otherwise they sink :~).


If you want to be super correct, you could mention the standard by name: https://en.wikipedia.org/wiki/ISO_3166-1_alpha-2


This should return a mutable copy of `copiedArray`


The RDD is fun to see :)


@tombh this is a pretty older version, please take a look at the final version of the code in the PR.


It looks like this is only called once; does it have merit as a utility function?


We're on a Facebook group now

Sent from Outlook Mobile<https://aka.ms/qtex0l>

On Mon, Feb 22, 2016 at 6:06 AM -0800, "tweekley49" <notifications@github.com<mailto:notifications@github.com>> wrote:

I just added you. I'll add you to group when you accept it.

Sent from my iPhone

> On Feb 22, 2016, at 8:26 AM, mariahcann1995 <notifications@github.com> wrote:
>
> I did I haven't gotten anything
>
> Sent from Outlook Mobile<https://aka.ms/qtex0l>
>
>
>
>
> On Mon, Feb 22, 2016 at 5:25 AM -0800, "tweekley49" <notifications@github.com<mailto:notifications@github.com>> wrote:
>
> Mariah, Adam added you on skype. Any chance you can accept it so we can get you in the group chat?
>
> Sent from my iPhone
>
> > On Feb 22, 2016, at 8:11 AM, mariahcann1995 <notifications@github.com> wrote:
> >
> > Hey guys it is due tomorrow night at 2359.... https://wiki.archivematica.org/File_Browser_Requirements look at this.
> >
> > -
> > Reply to this email directly or view it on GitHub.
> >
>
> -
> Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16239951>.
> -
> Reply to this email directly or view it on GitHub.
>

-
Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16240940>.


Why don't you use camelCase names here?


will be watting


Shim Explanation with example


well i did that,  but still not working for me.


Strange test name


A check is missing on the value of the characteristic_id column


Add blank line before func().


The idea is that these developer-focused libraries were never really plugins in the first place. They could be activated as a plugin, but were never intended to. The main problem is the lack of dependency management in WordPress. Having something as a plugin is not a good way to manage dependencies, even with something like TGM Plugin Activation, because it doesn't integrate with version control.

At the moment, using Composer or Git submodules is the best way to include these libraries.


Ok also GA seems to support having multiple trackers on same page so no
issues we can add on more for detailed stats on this alone
On 22 Feb 2016 21:49, "Julien Dubois" <notifications@github.com> wrote:

> Yes it's mixed we the other pages (that's the idea, we get the total
> number of page views on the website), but you can get stats on a specific
> web page, in this case "/jdl-studio", so you can have the individual views
> too
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/jdl-studio/commit/09a38e8ba9187e79e69f77517422b878d1fff7d0#commitcomment-16240487>
> .
>


Indentation


weird it wasn't there.


***Code Quality - Nitpick:***
Events could be converted to array before serving to the frontend


Closes #3


@vilchik-elena except if step 2 depends on the result of step 1.


If it panics isn't this an issue? Also, what testing was done with the race detector?


It should be 2_0_0.


Please remember that:
* `gpages_build.sh` is no longer copy-pasted everywhere
* travis needs a `.travis.yml` file to work


OK. Yeah, I went with composer already. It was just a bit of a surprise to see the plugins deactivating themselves because of invalid header - no mention of the transition in README.


I think it's still valid - the fit result was previously stored in a map passed in by the caller. Now it is stored in a new map returned from the function.


Ah sorry, it's in the changelog file but not the readme.


OK, tested kymations addon - and it does exactly what I had plans for in this branch.
Allows the shopowner to turn off/on navbar items, swap them around and so on.

My idea is pretty much the same.
I believe it can be done a little bit cleaner than the Addon, so I will carry on and see where it goes.

Thanks for the heads-up @katapofatico - appreciated much.


TeamCity libpalaso :: palaso-precise64-libpalaso-2.6 Continuous [Build 2.6.1051.38a7a5667cfc33502cb51f75ca3ea2fbee21210d](http://build.palaso.org/viewLog.html?buildId=69458&buildTypeId=PalasoPrecise64v26Cont) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:10:42


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.688.38a7a5667cfc33502cb51f75ca3ea2fbee21210d](http://build.palaso.org/viewLog.html?buildId=69460&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3427, ignored: 126 Build time: 00:11:05


It checks that the single row does not have characteristic id (id 10), so I don't think the condition should be added.


Actually, I did add mention of it to the readme under the 'Usage' section, but I didn't bother making it highly visible.


Yes. How accurately do we need wait 1 ms to call `btn.fall(&btn_onpress);` after we call `btn.mode(MAIN_BTN_PUPD);`?

The original code was 10 ms of waiting, so I probably should have left it at 10 ms, though. I'll fix that.


Oops. Half the point of those convenience triggers is to eliminate possible copy/paste-related errors, heh. Thanks. :+1:


Created initial README file


wat


```Resolv::IPv4::Regex``` and eventually ```Resolv::IPv6::Regex``` aren't going to match during PTR record creation -- foreman sends ip reversed, with 'ip-addr.arpa' appended to it.


I'm sure it was a bug fix from upstream. It's probably not often that only one dimension changes in the map.


onJoining, onMessage and onLeaving need to all take `Peer from, WebChannel to, String content` as per the specification. So Peer needs to be implemented.


asd


Yes, I had it working on Friday, I didn't push it yet because retina display shows some issues for unclear reasons right now :/
Otherwise it looks better.


The following doc(s) have been deployed to the [1366ghtem](http://1366ghtem-static-docs.pantheon.io/docs) Multidev environment:
- [/ISSUE_TEMPLATE](http://1366ghtem-static-docs.pantheon.io//ISSUE_TEMPLATE)
- [/PULL_REQUEST_TEMPLATE](http://1366ghtem-static-docs.pantheon.io//PULL_REQUEST_TEMPLATE)


An approach like [this](https://github.com/CyanogenMod/android_device_motorola_msm8226-common/commit/8594abf9e8941a71c2721156dde4c4c7036b66a0) is better


I changed my JS coding style recently(actually some months ago) because most of examples and conventions on web don't recommend using `,` for multiple variable definitions. I personally think that it makes diff ugly, especially when removing the top variable. Using `,` doesn't work well with ES6 module import as well.

I've postponed the change for PS as it's not essential, but today I had a little time and did it :smile:


Redundant https://github.com/CyanogenMod/android_frameworks_av/commit/a1f4fae05db0af7c1ab17d9566e6786f4bdcf7ff


Yes, I was thinking about that. I am not sure how you guys want to approach this.


You're right.


@pynicolas so will it depend to this example?
`"a" + foo("a" + b + "c") +"c" + "d" + e`  // should be 2 issues


`qName` seems mysterious. I guess "q" stands for "qualified"? If so, "qualifiedName" or "fullyQualifiedName" would be a better name of the parameter.


Why? It doesn't make much sense showing Preferred Network Type if there isn't any sim in my opinion...


Based on the latest discussions, I think it is good to mention that Hawkular Metrics will have a separate blog post about this.


Perhaps change this into (something like):

```
{% if name %}{{ name }} <{{ email }}>{% else %}{{ email }}{% endif %}
```


LGTM


The following doc(s) have been deployed to the [twootenpat](http://twootenpat-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/users/generating-ssh-keys](http://twootenpat-static-docs.pantheon.io/docs/articles/users/generating-ssh-keys)


TeamCity libpalaso :: palaso-precise64-lfmerge Continuous [Build 2.6.8.38a7a5667cfc33502cb51f75ca3ea2fbee21210d](http://build.palaso.org/viewLog.html?buildId=69459&buildTypeId=Libpalaso_PalasoPrecise64lfmergeContinuous) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:05:42


I've heard about some issues with FM Radio with this enabled. Did you test this?


Why does this repo need to include the source for picard? Can it not be downloaded at build time if the source is truly needed?


i'm online sir


Will the old one continue working for a while?


Hello,
I am having the same issue
https://i.gyazo.com/9604c9368320495b513b8c84f72ae236.png


i do not understand where is your correction ? someone suggest to replace clobbers by a merge to correct exposition problem of openDatabase, it's wright, but it is not commited even in the last repo.
is there a fork more recent ?


My concern was handling a partial or interrupted write. I figured write_all would do the work for me, and only return Err on a fatal error.


Immer noch im Master. Document anstatt Doucment.


Ebenfalls noch im Master.


this error will overwrite anything from line 42 or 44. which means we potentially have 2 points where our errors are swallowed.

lets fix that.


Yes, I'm not planning on removing any of the old ones. Just would like to use a different one for each version.


It should handle QStringList just fine, no?


Couldn't say it better!  :)


Bug wrong end_date conditional in payment history.
As a result the payment history is not showing any purchase made in the current time frame.

Examples

http://take.ms/r1YGk
http://take.ms/AXpH6


Yes, unless some more important loglevel than CRITICAL exists. That should make it very clear to users that something is wrong.


Simply took the getTotalLength function outside of the skillsAnimate function.


Can you make a pytest test?


> it is not clear to me what is the intention of the `yield` + `throw`

The enumerable turns into a state machine. Since this specific `yield` is giving up a *specifically configured* path, if control returns to the state machine to get another, that means that the *specifically configured* path has failed. So we should then throw rather than continuing the search.

That said, that can be made more self-evident with a comment, which I will add.


Зачем ты продублировала проект?


change to: "executors must set CPU resources to at least 0.01 and memory resources to at least 32MB.


It's really important to clarify that when a unit is stopped and another started and they have an ordering then it doesn't matter which one they have the stop is always done first, and the start second. This part has gone missing in your work.


@gforcada Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.recipe.codeanalysis/10//violations
```
plone/recipe/codeanalysis/__init__.py:172:13: T001 print function found.
plone/recipe/codeanalysis/__init__.py:187:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:196:9: T001 print function found.
plone/recipe/codeanalysis/__init__.py:244:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:59:9: T001 print function found.
plone/recipe/codeanalysis/analyser.py:66:13: T001 print function found.
plone/recipe/codeanalysis/tests/test_csslint.py:24:0: P103 other string does contain unindexed parameters
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


use MB instead of mb


Without further data, this seems like an error on your side. Technical support for Tunebot is by payment.


Electric-Service-Refactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed, 0 test warnings)

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


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.697.38a7a5667cfc33502cb51f75ca3ea2fbee21210d](http://build.palaso.org/viewLog.html?buildId=69461&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:07:06


I think we'll have a better path forward soon using ESLint https://github.com/PolymerElements/polymer-starter-kit/pull/708. The reporter is much better and produces actual sensible output.

Good to know about Browsersync. I didn't realize the linter was tripping it up. I think we can look into adding ESLint back into the flow once in a way that doesn't crash serving once we have the recipe landed and folks have had time to play with it.

> Am not at all impressed...

I don't think comments like this are helpful. Please remember everyone involved is doing the best they can. OSS is a labor of love :heart:


replace "Set" with "The"


@pynicolas  Intuitively I would pass `body.statements()` instead of `ImmutableList.<Tree>of(body)`. Why so?


[Uploading 11.zip…]()


"... which means a double value should be used."


Nice reduction of duplicates! :+1:


Mexican.


`this.setState({ ...this.state, ...patch })` is different to `this.setState({ ...patch })` ?


@SimonCropp Previously is the session ID was unequel the ready message was dropped. It now only drops the message if the worker is disconnected. Do a split view, the changes currently look a bit weird in a unified view. Then you see that the first if statemen is removed.


seeing this, we can probably export an object `loggers` or `log`.

```js
export const loggers = {
  prettifier: debug('prettifier'),
  reviewState: debug('prettifier:review state'),
  labels: debug('prettifier:labels'),
  http: x => debug('http')(JSON.stringify(x, null, 2))
}
```

Maybe in another PR


Этот файл в удаленном репозитории не нужен.


You can use the map find optional return value to not have to do a string check:

```
apiPolicyID, policyMatchFound := baseSession.PolicyPerAPI[t.Spec.APIID]
	if policyMatchFound {
        ...
```

:-)


My understanding is that this is an anti-pattern in React.

this.state.{x} should be used along with setState() when changes should trigger a rerender...
this.{x} should be used for state that has nothing to do with a render

this.props.{x} should not be saved as local state, including computed properties, but should be used directly. Otherwise, there is no single source of truth.

The only exception is when the prop is used to set an initial state that will intentionally end up out of sync. Is that what you are doing here? I'm not sure, and don't want to say the wrong thing if I'm missing why you set these properties.


Classes/Eat Now/Detail View Controller/EateriesGridViewController.swift

What I meant by diacritic insensitive search is that it is a string comparison option.
```
var s = "sfsfsÈ"
s.rangeOfString("e", options: [.CaseInsensitiveSearch, .DiacriticInsensitiveSearch])
```
This will still find the range of the capital È as the "e". If you can change these search options and remove the additional cases in the JSON that are there simply to avoid diacritics, that would be great!


abc:p@ssword@github.com leads to fatal error connecting @ssword@github.com


du idiot hast 0 und nicht r geschrieben das hat uns jetzt ne Stunde arbeit gekostet <:(


I mean it says TNewsNotebook instead of TNewNotebook in the comment


Yep, this is the problem. With this code, the point of the whole function is useless;)


gitignore did not work


@robdodson Apologies, didn't mean to come off so crabby! Mainly just emphasising that, given how PSK is now integrated into things like `generator-polymer`, removing linting altogether adds quite a bit of additional labour at the start of every project for those of us who require linting.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1324.9614d1000c9f0c7ab67176feeff8a36d333cdb61](http://build.palaso.org/viewLog.html?buildId=69463&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1016, ignored: 12 Build time: 00:08:07


init commit


Alright, thanks for the info


du idiot hast 0 und nicht r geschrieben das hat uns jetzt noch zusätzlich ne Stunde arbeit gekostet <:(


@vilchik-elena changed in edbc11e


no, but in this case I'm using `set` from `lodash` to generate the new object that i then pass to `this.setState`.
You probably missed it as this code is a bit compressed...written on two lines it would be:
```js
const newState = set({ ..._this.state }, path, newValue);
this.setState(newState);
```


Ah alright, this by chance follows AirBNB's JavaScript style guide. See the [Variables sections](https://github.com/airbnb/javascript/tree/master/es5#variables)


I have added an additional tracker and have given you admin access for the
same so now we have stats added to main website as well as individual stats

Thanks & Regards,
Deepu

On Mon, Feb 22, 2016 at 10:52 PM, Deepu K Sasidharan <d4udts@gmail.com>
wrote:

> Ok also GA seems to support having multiple trackers on same page so no
> issues we can add on more for detailed stats on this alone
> On 22 Feb 2016 21:49, "Julien Dubois" <notifications@github.com> wrote:
>
>> Yes it's mixed we the other pages (that's the idea, we get the total
>> number of page views on the website), but you can get stats on a specific
>> web page, in this case "/jdl-studio", so you can have the individual views
>> too
>>
>> —
>> Reply to this email directly or view it on GitHub
>> <https://github.com/jhipster/jdl-studio/commit/09a38e8ba9187e79e69f77517422b878d1fff7d0#commitcomment-16240487>
>> .
>>
>


если введено не число упадет exception


Not sure I think it's necessary, it's pretty compact code already. Leaving it for later 'optimization' :)


The sources could definitely be downloaded at Docker build runtime, you are obviously right. This was the quick and dirty way to get it running. Still, I have pushed an even more simplified solution, which is pulling the semi-official image from the Broad and adding a CMD entry so that it is compatible with our CWL workflows.


Maybe group this in an object called `labels` or similar, to better convey their purpose.

```js
const labels = {
  macro,
  wip: 'WIP',
  inReview: 'in review'
};
```


В меню сказано что 1 - это чтение из файла.
Падает NullPointerException


I think most people know the consequences of using plaintext.


correct, and "falsehood" is more consistent with your later text...


На добавлении нового элемента тоже падает NullPointerException


Would it be to exclude read-only properties in the constructor, or to only include read-only properties in the constructor?

I think the term "readonly" will possibly lead to confusion, since that's actually a keyword in C# and has a different meaning than an immutable property.

I think mixing read-only and required properties makes things a little more complicated. From a strict OOP perspective, required properties would be in the constructor. But what about read only properties that aren't required? Those properties would need to be deserialized, but immutable. I'm not sure the best approach, but it seems difficult to solve only in the constructor.


Is this file supposed to be here?


Notes: test-ci is not integrated. Right now the sequence of events is:
![image](https://cloud.githubusercontent.com/assets/8346437/13222312/b1ffd948-d97f-11e5-9e3d-ba8e89dc75f6.png)

We could more closely tailor our build process using scripts with this: https://docs.npmjs.com/misc/scripts


This will be very helpful. Great addition!


nice. this makes a lot of sense to me


ok, sorry about that.


O'Brien Christian, February 22 2016, 10:17AM


have you considered de-structuring?

```js
const [match, issueNumber] = /\(closes #(\d+)\)/.exec(pull.title)
```


![capture d ecran de 2016-02-22 16-16-26](https://cloud.githubusercontent.com/assets/1388/13222344/e2d72fb2-d97f-11e5-8138-8eff2710ccc8.png)


It wasn't really serving any purpose. The variable name contains the same information, and nothing is ever done with the name. Of course, I didn't consider debugging. That said, you can always use the callbacks to aid debugging.


Ok. (I didn't group them because we were already in the context of `labels.js`)


@gophry , thanks for noticing. Can you please report an issue instead?

The https://github.com/rocketeers/rocketeer/issues/633 seems to be related.


From the [spec](https://github.com/OAI/OpenAPI-Specification/blob/master/versions/2.0.md#fixed-fields-13)
```
Relevant only for Schema "properties" definitions. Declares the property as "read only". This means that it MAY be sent as part of a response but MUST NOT be sent as part of the request. Properties marked as readOnly being true SHOULD NOT be in the required list of the defined schema. Default value is false.
```

So to ensure that it's not part of the request, we must not allow it in the constructor.


Yeah, I get it. However it wasn't immediately clear what those const were for.


@glebm thanks. Done 2ce3648


marco? Who's marco?


Задание 2.1 не принято.

Загрузка из файла не работает (так как он прописан в коде) и добавление новых сообщений тоже (NullPointerException).

К тому же завтра дейдлайн по Clean Code, а потому рекомендую все-таки почитать как нужно оформлять код в Java. Пока это выглядит как незачет по 2.2 в том числе.

Удали лишние файлы и дубликаты.


> *username* and *password* are credentials for SASL authentication. It requires support in libmemcached, and `binary=True`.

Maybe? I think the current way is quite wordy for something that is readily expressible in a single paragraph. :wink:


Nope, I never remember I can de-strctur arrays too.
Still, `/blabla/.exec(string)` might return `null`, so de-structuring is not possible. (`drop` always return an array)


Yeah, I'd prefer not to include the entire source in the dockerfiles repo, for several reasons. I'm going to close this PR but please submit a new one for the other tools and I'll be happy to merge it and create automated builds


+1 And also what about osx and others which uses this runtest.sh? Instead of hard-coding for linux, it should start from "Microsoft.NETCore.CoreDisTools" to redirect the right one per platform.


Perhaps this

> *username* and *password* are credentials for SASL authentication. It requires support in libmemcached, and `binary=True`. Test for local support with `pylibmc.support_sasl`.


added Profile/Conversations fragment functionality


The following doc(s) have been deployed to the [cilogs](http://cilogs-static-docs.pantheon.io/docs) Multidev environment:
- [scripts/deploy-live.sh](https://github.com/pantheon-systems/documentation/commit/c3136990cd909f03fc304cede0d7f7a9211c18d5/scripts/deploy-live.sh)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/c3136990cd909f03fc304cede0d7f7a9211c18d5/scripts/deploy-multidev.sh)


Can you make a new branch without these commits?


And yes, even though Exodus-1 has 7K changes, I've verified that nothing's broken, and no tile states have been removed.


That would work better for me. Can you send a link?

> On Feb 22, 2016, at 9:50 AM, mariahcann1995 <notifications@github.com> wrote:
>
> We're on a Facebook group now
>
> Sent from Outlook Mobile<https://aka.ms/qtex0l>
>
>
>
>
> On Mon, Feb 22, 2016 at 6:06 AM -0800, "tweekley49" <notifications@github.com<mailto:notifications@github.com>> wrote:
>
> I just added you. I'll add you to group when you accept it.
>
> Sent from my iPhone
>
> > On Feb 22, 2016, at 8:26 AM, mariahcann1995 <notifications@github.com> wrote:
> >
> > I did I haven't gotten anything
> >
> > Sent from Outlook Mobile<https://aka.ms/qtex0l>
> >
> >
> >
> >
> > On Mon, Feb 22, 2016 at 5:25 AM -0800, "tweekley49" <notifications@github.com<mailto:notifications@github.com>> wrote:
> >
> > Mariah, Adam added you on skype. Any chance you can accept it so we can get you in the group chat?
> >
> > Sent from my iPhone
> >
> > > On Feb 22, 2016, at 8:11 AM, mariahcann1995 <notifications@github.com> wrote:
> > >
> > > Hey guys it is due tomorrow night at 2359.... https://wiki.archivematica.org/File_Browser_Requirements look at this.
> > >
> > > -
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > -
> > Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16239951>.
> > -
> > Reply to this email directly or view it on GitHub.
> >
>
> -
> Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16240940>.
> —
> Reply to this email directly or view it on GitHub.
>


Sorry, I had it commented but took it out. I'm explicitly doing the superfluous math to strip it of units. So `800px` becomes `800`.


Une version doit être compilé ?


yeah, I just mean if the variable is the ratio itself, makes things a little tidier, imho. then you can just do 1 / ratio * 100%


```js
const [match, issueNumber] = pull.title.match(/\(closes #(\d+)\)/) || [];
```

- not a fan of `exec`
- not a fan of extra lodash


Thanks. Will implement a Stylus version for ya. =)


Globally setting "LD_LIBRARY_PATH" seems not ideal. I'm not sure how gcstress tests work, but ideally we just put this binary in the same directory as the one that consume side by side, and dynamically search this in the consumption side.


Non ce n'est pas vital


Oh we've had it in Axis for a long time. :)

ps you should check axis out again because
1. ) Added tons of stuff you will love
2. ) Its going to get converted to postcss


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16644](https://teamcity.mesosphere.io/viewLog.html?buildId=141111&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:03:45


ThermCalcRelHumUseSameHumidRatioDryBulb (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (2102 of 2104 tests passed, 5 test warnings)

 * 3 tests had: AUD diffs.
 * 2 tests had: ESO big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 2 tests had: ERR diffs.

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


HHH


And did a quasi conversion to scss here (which has some other stuff axis doesnt)
https://github.com/kni-labs/knapsack/tree/master/knapsack


of*


I had removed this comment because it does not really make sense here, it was probably copied from another location. I think it would be nicer if we extract this logic into a helper function like `accessProjectModel(Action)` and have a proper JavaDoc comment on that one.


the retry stuff might be too much....?


This causes a problem with the getFileSrc function in UploadService, because it adds another forward slash after the base URL.


I totally agree that `isDeepTruthy` is a joke. I refactored `didFilterPass` [here](https://github.com/dylnslck/restle/commit/c7413df6e429b9b29b9dabadf607194f5af09511).


I'm wondering why the files list is in here at all?


@aendrew no worries. I think we all definitely want to add it back. We had a surge of issues related to it and JSCS a few months ago and it was easier to remove it and let the power users add it back than force it on everyone while we figured out the right approach. I think once we land the ESLint recipe and all the maintainers have had a chance to give it a shot we can look into rolling it back into the gulpfile. We can probably iterate on this pretty quick by adding it in as a commented out option (as you mentioned).


This isn't a full sentence.  What is the documentation trying to say here?


Not sure you can have 2 trackers, but you don't need that second tracker: it's just a filter on the "/jdl-studio" URL in the main tracker. I can even make a custom report if you want.


Same thing. You could also do

```ruby
like = Like.new(bookmark: bookmark, user: current_user)
```

None are more correct.


**[unitTests_php_branches run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_branches/27836/testReport/) for this commit**


commented last line


It is not misaligned.
Previous code was available for SPI2 and SPI5.
The new code is for SPI1 to SPI6. The 'if' was turned into a 'switch case', depending on the APB1 or APB2 clock to be used for the current SPI.

There is no plan for dynamic calculation.
Cheers


Discussed internally and we have concluded that for now because we do not have aliases, yet ; we will just keep the device-type nuc.


previous comment


Cool! I was thinking there is a central component here that would be pretty easy to extract and make it possible to define arbitrary themes to install using a simple DSL. Would you have any interest in working on that? I could keep you in mind and if I start that project I'll drop you a line and let you know.


Please remove the lul test ^^


super-cool, I never thought about enumerable and yield to create such a thing, lovely.


Hmm, that's really weird... it should at least show you your IP addresses...

Any idea you can gdb through this quickly and check what it causing this? But please consider applying #2685 first, that changed this code a lot.


**[unitTests_php_contractual_responsibilities run failed](http://jenkins.wikia-prod:8080/job/unitTests_php_contractual_responsibilities/432/testReport/) for this commit**


Charlie - how do I get on to edit?


look like here memory leak.
may be in other returns in this function too


Variable acro_yawRate_max seems to be unused here?


What do you think of this? https://github.com/miykael/nipype-beginner-s-guide/blob/master/homepage/images/voxel_new.png

And there is a small typo in the last word of this commit "voxels".


TeamCity libpalaso :: palaso-precise64-master Continuous [Build 3.0.698.280149c5d8ae19c0c0fe7ee1ead761ab0971f2f5](http://build.palaso.org/viewLog.html?buildId=69465&buildTypeId=bt322) outcome was **SUCCESS**
Summary: Tests passed: 3407, ignored: 107 Build time: 00:06:52


TeamCity libpalaso :: palaso-win32-master Continuous [Build 3.0.689.280149c5d8ae19c0c0fe7ee1ead761ab0971f2f5](http://build.palaso.org/viewLog.html?buildId=69464&buildTypeId=bt223) outcome was **SUCCESS**
Summary: Tests passed: 3427, ignored: 126 Build time: 00:07:35


coala -> coala-bears


This doesn't work - we don't want multiple identical keys..


Ah okay, yeah I'm dumb. Will update in a sec for 2.0.0.


I did a new "jdl-studio last week" shortlink on the GA website, can you see it?


didn't you want to avoid those?


ok, but to calm `eslint` we have to add also `// eslint-disable-line no-unused-vars` as `match` is not going to be used


iteration (evanweaver) - i386-Windows-7-VisualStudio-12: Build Failed

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


Which would be better '' or EMPTY?


> Script named 'npc_netherweb_victim' does not have a script name assigned in database.

The script is located here: https://github.com/TrinityCore/TrinityCore/blob/3.3.5/src/server/scripts/Outland/zone_terokkar_forest.cpp#L304


nice


line 1,2,17 : it will be linked to our proprietary libril, not to open-source libril(hardware/ril)


at least until https://github.com/coala-analyzer/coala-bears/issues/77 is resolved


this is needed on 6.0 MM build environment, shouldnt be reuired on <6.0


are you sure? I thought that case is ignored by eslint because array destructuring is positional


iteration (evanweaver) - Win64-Windows-7-VisualStudio-12: Build Failed

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


I thought it too, but it's complaining in my editor so yes


What is the purpose of this method? Should this be returning a boolean and be called `liked?(bookmark)`?

A better way to do this, so Rails can cache the data, is

```ruby
def liked(bookmark)
  # uses the existing association rather than new sql calls
  likes.find_by(bookmark_id: bookmark.id)
end
```

Although if I understand the intention here, it should probably be:

```ruby
def liked?(bookmark)
  likes.where(bookmark_id: bookmark.id).count > 0
end
```


or maybe it is ok... not sure, although it should be enough to have just a list...


oh, I found how.

```js
const [, issueNumber] = pull.title.match(/\(closes #(\d+)\)/) || [];
```


The less files are compiled into CSS, and not included in release packages. If you want to use this before we've made a new release, you'll need to run the development version - there are instructions here:

http://jupyter-notebook.readthedocs.org/en/latest/development_js.html


@gabro I'm editing also this one


/startcomponentcleanup option is not available on win7 and win2k8
we need to skip this part when on these windows versions


@NathanWalker Thanks for your reply.

We are using systemjs. Is dragulaModel working fine for you?


:bow:


That would be awesome @vcavallo but I must say that my skills are limited besides bash, C++ and CSS.


You have added a multiplication to inside a loop... is this really an optimisation?


You should comment the test cases we have not actually seen as such


I just want to make note that even though this program was tested in the console I changed the console outputs to alerts for preference.


I thought about it, but is it valid? I tried it on `node` a couple of minutes ago and it didn't work properly..


sysprep needs to be done after the diskpart/defrag has ended, as it will shutdown the system


So where does this leave us now? Are you proposing to create a new `ResumptionSupportingConnectionStore` interface extending `ConnectionStore` that would include these additions? Or should we turn `ConnectionStore` into a class and support different strategies for persisting Session state only instead? Maybe you are right and nobody wants to deal with the complexity of managing the Connections when they are only interested in getting the session state persisted somewhere.
However, the more I think about the clustering and fail over use cases the less convinced I am that DTLS session fail over between nodes makes any real sense. In particular I think that the DTLS spec is not very precise regarding how session resumption is supposed to work and I suspect that we are currently not doing it right because we do not consider the current connection state's epoch and record sequence numbers...


variable name should be capitalized for llvm style. same for 'zero', 'pCmp', 'pCast'


Video recording is working fine even without this.


Can't find this property anywhere, I belive this is Xiaomi specific.


this flag value check is not necessary


ok bro thank you so much :+1:  ı released the source, you can take a look too. feel free to modify :+1:  https://github.com/sirmordred/rild/commit/268e5837124a94b3a29ccb14004f244922258f35

also ı can say that f*king old rild(so bionic hacks)  is only thing that block us from official cm :)


not needed


Embarrassing!  Thanks :)


On line 57, you should undo that change.  It should read debugme cat target.log without the $()'s.

Otherwise the changes look good to me.  I am okay with this being placed into staging for testing.


pls english!!)


Great point. Also, I was mistaken about bufwriter, looking at the write traits' sourcecode, write_all actually doesn't call flush, so this is perfect as it is.


Might want to make constructing this key a helper function?


Nice :+1:


sollten wir hier nicht besser utc nehmen anstatt die system zeit des servers?


Although, now that I think about it, completely flattening an array/object could be more computationally intensive than `isDeepTruthy` as long as `isDeepTruthy` short circuits when a falsy value is found.


still git ignore issue not fixed


The following doc(s) have been deployed to the [527rsyncl](http://527rsyncl-static-docs.pantheon.io/docs) Multidev environment:
- [docs/articles/sites/logs/downloading-live-error-logs](http://527rsyncl-static-docs.pantheon.io/docs/articles/sites/logs/downloading-live-error-logs)
- [docs/articles/sites/logs/mysql-slow-log](http://527rsyncl-static-docs.pantheon.io/docs/articles/sites/logs/mysql-slow-log)


Electric-Service-Refactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1005 of 1005 tests passed, 0 test warnings)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-100.0%25-green.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-29.01%25-green.svg)


Copy&paste error, `LongError` should be here.


Copy&paste error, `LongWarning` should be here.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16645](https://teamcity.mesosphere.io/viewLog.html?buildId=141117&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 235 Build time: 00:08:50


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `bears/tests/go/GoReturnsBearTest.py`, line 34.*

The issue can be fixed by applying the following patch:

```diff
--- a/bears/tests/go/GoReturnsBearTest.py
+++ b/bears/tests/go/GoReturnsBearTest.py
@@ -31,5 +31,5 @@

 GoReturnsBearTest = verify_local_bear(
     GoReturnsBear,
-    (good_file1,good_file2,),
+    (good_file1, good_file2,),
     (bad_file1, bad_file2,))
```


**Shortlog of HEAD commit is too long.**
*GitCommitBear, severity 1, section `commit`.* *Project wide.*


Корзины на главной нет. по ховеру ничего не отображается.


closes #5


Все так же закрывается выпадающий список при попытке на него навести.


RFIFOP(fd,2) already in playername, probably here better use:
sd = map->nick2sd(playername);


I don't like too much lazy stuff...  (because of thread and locks and unnecessary perf issues).

I looked into this over the week end and couldn't understand why configuration.addInputStream() was not registering the named queries. Were you able to understand why?

Thanks for fixing this issue!


ack


This is testing commit


Good point.


Same here.


@rodfersou FYI


As reported in [closed issue ](https://github.com/yukuku/telebot/issues/36) simsimi could easily work again


I think we should use relative links for pages on GOV.UK i.e. `/government/publications/...` not `//gov.uk/government/publications/...`. I know that there are a few other places where this is not observed but I think it is the right approach and the approach that is used in most cases.


Can't find this property anywhere.


Там дуже руського нету, а ты ещё и украинский захотел :D


Раз уж начал делать лейблы, то плейсхолдера не будет. В инпуте спозиционирован label.
Определить что инпут не пустой можно вот этими правилами:
input:not(:placeholder-show) {
  /** input is not empty */
}

input:placeholder-show {
  /** input is empty */
}


Why are we sleeping inside the initialization of a command?


@markelog done


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16646](https://teamcity.mesosphere.io/viewLog.html?buildId=141131&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:04:12


very dodgy?


As per google docs its possible and I tried and it works Im able to see
realtime tracking on both GA
https://developers.google.com/analytics/devguides/collection/analyticsjs/creating-trackers#working_with_multiple_trackers

Btw I dont see the shortlinks, I also dont see the custom reports btw, may
be those are per user settings

Thanks & Regards,
Deepu

On Mon, Feb 22, 2016 at 11:33 PM, Julien Dubois <notifications@github.com>
wrote:

> I did a new "jdl-studio last week" shortlink on the GA website, can you
> see it?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/jhipster/jdl-studio/commit/09a38e8ba9187e79e69f77517422b878d1fff7d0#commitcomment-16243344>
> .
>


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16648](https://teamcity.mesosphere.io/viewLog.html?buildId=141133&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:04:18


Что это и зачем?
Не засоряй репозиторий тем что еще не актуально.


`like =` or `like ==`?


replace with

"../../click2vox/javascripts/widget-0.3.js"


also ı found out rild-libargs property in system.prop doesnt do anything (https://github.com/arco/android_device_samsung_ancora/blob/cm-12.0/system.prop#L5 ) because ı digged our proprietary rild with IDA and found that this property is hardcoded to ttyS0
and thats why i'm getting the same(ttyS0) from "getprop rild.libargs" in terminal.


@thebigsmileXD so:

long-click boat to get in => action=2
click "enter" button for boat => action=3?


Looks like a mistake:
Replace `if (NOT GIT_EXECUTABLE_NOTFOUND)` by `if (GIT_EXECUTABLE)`


I didn't know about this property. It was added in iOS 9, I'm reading up on it now.

Very nice!


Ah, this is from the curriculum. I can see why you were confused. I read through this last week before you started on it, and missed it until I saw the actual code.


Congratulations!!!


here size error item_monster_name size is size of str field, but struct packet_gm_monster_item size include str field + packet id
Probably need:
safestrncpy(item_monster_name, p->str, sizeof(p->str));


Approved


For reference: #28


sdfasdf


The type could also be `Action :~> ExceptT ServantErr IO`, but I don't have any preference.


and also sorry to disturb you @arco but may ı ask why did we use this https://github.com/arco/android_device_samsung_ancora/blob/cm-12.0/BoardConfig.mk#L99 altough we are not using source-built libril.rild stack ?


Checkout the commit message. I think it would if we can find a good way to do so.


I believe there is a type in this fix, line 1354 should read

`return instance.hookValidate(options).**then**(function (err) {`

because otherwise this changes the behaviour of validate: true, because it means that it will create all records that succeed in a bulkCreate instead of the normal behaviour if validate:true is set of rejecting all of them.


We want to avoid silent downcasting, but rather than having to deal with too many dtypes in the C code, it is better to upcast to int64.


Well, you could do something like:
```C
if (PyArray_DESCR(%(size)s)->type_num != NPY_INT64) {
    odims[i] = ((npy_int64*)(PyArray_BYTES(%(size)s) + PyArray_STRIDES(%(size)s)[0] * i))[0];
} elif  (PyArray_DESCR(%(size)s)->type_num != NPY_INT32) {
    odims[i] = ((npy_int32*)(PyArray_BYTES(%(size)s) + PyArray_STRIDES(%(size)s)[0] * i))[0];
}
```


I believe the conclusion was the other way around.
The .coffee file needs to reflect the yocto machine name
@agherzan , confirm?


I guess you forgot to update the osgi version: https://github.com/mockito/mockito/commit/aecdcf52d67ae627e18d84db4e45627d74578702 ;)?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16647](https://teamcity.mesosphere.io/viewLog.html?buildId=141132&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:09:02


This file should not have been committed.


this is cool, but I think we should just validate all text fields to be less than 255 - BT validates lengths of lots of stuff, including addresses


Same thing here.


`ProcessRequest` only needs to return an error and code value. `applyRateLimiting` will handle rate limits and quotas and apply those to the key that has been supplied (in this case it would be the `perAPISession`, which is what we want, as that is what needs to be updated - see the `updateSession` calls in the `applyRateLimiting` method).

So, theoretically, at this point we have retrieved the base session, realised that we need to actually use rules from the api-specific one to control access, and then apply that to the session store (redis), since that's where the data needs to be kept in lock-step across requests.

The problem we have here is that [later in the request cycle, we write the current quota limits etc to the header of the return response](https://github.com/TykTechnologies/tyk/blob/master/tyk_reverse_proxy_clone.go#L495) so the client knows it's quota status. On a per-API basis this will change depending on the endpoint, but since we are not actually using the temporary API session throughout the request cycle here (only selectively) the wrong values will be written to the header.

I think though, we could safely move the quota header write to this `processRequest` method here since this is where the quota limit is written to the DB, so we may as well write it directly to the `http.ResponseWriter` since that's all the proxy responder does.


remove .sass-cache too and gitignore it


How does this ever complete if we are not checking the switches in isFinished?


here can be buffer overflow. You cant trust to field p->PacketLength, it can be very small or very big, bigger than buffer.
Probably before need check this field and return from function if it wrong


This too. I can see why you were a bit confused.


I know, It is my fault for editing directly in GitHub and pushing commit without filling the form.
Do you want me to amend it?


My sincere congratulations !!


This is the solution i provide for filtering. This filter fits all the requirements, but i'm not 100% confident, it need to be tested a little bit more, mostly because there a lot of possible usecases, and i tested it just with the few that jump in my mind.


Hi,

I installed from the released tar.gz / setup.py and got errors regarding JSONDecodeError in python 3.2. I have simplejson latest version and had to work arround like so to make it work :
- from json import JSONDecodeError
+ import json

Replace every JSONDecodeError by ValueError which should still call JSONDecodeError.


If these comments are useless, remove them.


Кроме того, вас просят доказать, что сумма всех чисел натурального ряда до n включительно равняется n(n+1)/2, а вы показываете что-то совершенно другое.


From a glance my version is better than both those. Feel free to use it if you'd like.

Ironically Jeff correctly coerced me into liking modular tools. :)


The picture looks perfect.  Typo corrected.  Requested new pull.


@pynicolas Is  parameter name `conditionBlock` good here? I would expect `successor`


@dallisg711 check out this paper regarding the idea for a "design space" paper for motion washout: https://www.researchgate.net/profile/Ruud_Hosman/publication/228889776_Integrated_design_of_flight_simulator_motion_cueing_systems/links/54292ced0cf2e4ce940c892e.pdf

we can (should) include some of this in the lit review wiki as well.


:D


I believe you can cast size pyarray with [PyArray_Cast](http://docs.scipy.org/doc/numpy/reference/c-api.array.html#c.PyArray_Cast).


There should NOT be ANY file under /target.
In fact, this directory should not even appear. Please, remove it from the index and amend your commit so that binaries and others do not appear in the git history.


Again - Huh?  Why is this sleeping?


you have to remove phantomjs and add phantomjs-prebuilt": "~2.1.4" then upgrade karma-phantomjs-launcher to  "karma-phantomjs-launcher": "~1.0.0". Don't forget to delete your old node_modules directory because of caches


yeah, its about to get a lot more modular.

I like what you did with the `> *` probably going to add that to both.

Still trying to wrap my head around if a default background color would every be problematic.


Ah I knew I had forgotten to do something - I did this on my local version installed in eprints, but not on my cloned copy of this repository!


ah, fair enough.


@SalmonGER when you hit often enough or with a strong sword, yes (they regenerate 0.1 hearts every tick, like mcpe)

And combined with my latest commit they should drop the correct boat and despawn instantly


I am also having a garbage issue with slightly longer wires on the text lcd so my question is related to the below section. When you are moving or re-assigning the 46/47 and 42/43 pins are you having to physically cut and wire from the RADDS LCD connector to those pins on the Due header? I would like to try this out.

"When connections are ok with Ramps-FD, I got some issues with RADDS, the schematics suggest to use pins 42/43 for RS and EN but for me it is not working. I have tried on both RADDS I have and initialization is ramdomly failed or give a garbage screen. I got same behaviour with text LCD. I suspect some "noise" due to proximity of VCC so I moved RS/EN connection to pins 46/ 47 because Graphical LCD does not need and bingo - no more issue, same for text LCD using 42/43 for LCD 6/LCD 7 and 46/47 for RS/EN also solved the problem."


ma sei un grande!!


But if I head to do a bower/npm install for every mixin I need on every project it'd be nuts.


last note : after these steps all fails will be gone except chat module tests : socket.io.client.service.tests.js and chat.client.controller.tests.js  we need to rewrite this 2 or remove their use strict


Hmm, the method (`CheckSessionAndIdentityForValidKey`) that does the per-api session policy check will create it if it can't find a session that uses this policy, so it *must* exist if the request has got this far.

If the per-api session is suddenly gone, then something fishy is going on (it has been deleted manually), in which case the request should fail.

Also, in terms of intent, if an admin created a key with a per-api policy, and that per-api policy is not being found or applied, the admin should know about it, and the best way to force that behaviour is to have the request fail, otherwise, if it's a  passthrough, the key would continue to work, the policy would not be applied (instead a base policy would, which might be wrong or too generous).

I think failure would be the default here?


LGTM


Confirm. Needs filename change,


Do we need this here if it is in the base session?


Another paper to look at:

https://www.researchgate.net/profile/Max_Mulder/publication/267303028_A_Cybernetic_Approach_To_Assess_Flight_Simulator_Fidelity/links/5454e8f70cf2bccc490cc5f2.pdf

http://www.lr.tudelft.nl/fileadmin/Faculteit/LR/Onderzoek/Onderzoeksinstituten/SIMONA/Projects/Handling_Qualities/doc/aiaa-2003-5679_1_.pdf


Yes I understand, but don't we prefer to upcast in the perform/c_code rather than by adding a node in the graph?


> I looked into this over the week end and couldn't understand why configuration.addInputStream() was not registering the named queries. Were you able to understand why?

It does not load anything because it has no idea what to load: the path is set (and so the configuration loaded) when hibernate store is initialized at first request while HqlQueryExecutor component is initialized during application startup (did not really digged more) so it end up with an empty configuration.


The new axis will just be a collection of individual post-css plugins


@PEMapModder to get in is action 1, and yes, the button is 3


what's your Facebook name

Sent from Outlook Mobile<https://aka.ms/qtex0l>

On Mon, Feb 22, 2016 at 7:23 AM -0800, "jaybustah" <notifications@github.com<mailto:notifications@github.com>> wrote:

That would work better for me. Can you send a link?

> On Feb 22, 2016, at 9:50 AM, mariahcann1995 <notifications@github.com> wrote:
>
> We're on a Facebook group now
>
> Sent from Outlook Mobile<https://aka.ms/qtex0l>
>
>
>
>
> On Mon, Feb 22, 2016 at 6:06 AM -0800, "tweekley49" <notifications@github.com<mailto:notifications@github.com>> wrote:
>
> I just added you. I'll add you to group when you accept it.
>
> Sent from my iPhone
>
> > On Feb 22, 2016, at 8:26 AM, mariahcann1995 <notifications@github.com> wrote:
> >
> > I did I haven't gotten anything
> >
> > Sent from Outlook Mobile<https://aka.ms/qtex0l>
> >
> >
> >
> >
> > On Mon, Feb 22, 2016 at 5:25 AM -0800, "tweekley49" <notifications@github.com<mailto:notifications@github.com>> wrote:
> >
> > Mariah, Adam added you on skype. Any chance you can accept it so we can get you in the group chat?
> >
> > Sent from my iPhone
> >
> > > On Feb 22, 2016, at 8:11 AM, mariahcann1995 <notifications@github.com> wrote:
> > >
> > > Hey guys it is due tomorrow night at 2359.... https://wiki.archivematica.org/File_Browser_Requirements look at this.
> > >
> > > -
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > -
> > Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16239951>.
> > -
> > Reply to this email directly or view it on GitHub.
> >
>
> -
> Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16240940>.
> -
> Reply to this email directly or view it on GitHub.
>

-
Reply to this email directly or view it on GitHub<https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16243034>.


Ah, I hadn't checked the RIOXX guidelines and was simply going on what repository administrators had suggested. It makes sense to implement this way round though!


I think you may have misunderstood the meaning of `with`. It brings into scope the attributes of its argument. See <http://nixos.org/nix/manual/#idm46912467576704>.

An optional list can be written as follows:

```nix
(stdenv.lib.optionals netcdfSupport [ netcdf hdf5 curl ])
```

I also suggest that the flag to control NetCDF support have a name ending in `Support` (as many such flags in various other packages). The two names `netcdf` and `netCDF` are way too similar…

By the way, what are the reasons for this flag to default to false?


Yeah I tried that, but the tests for some of the controllers didn't pass as you may have noticed, so I left those for a separate PR so this could be cleaner and not have any blocking issues for other PRs. If you want to handle that in #1207/#1213, then that's perfectly fine.


TeamCity Glyssen :: Glyssen-master Continuous [Build *.*.1325.9614d1000c9f0c7ab67176feeff8a36d333cdb61](http://build.palaso.org/viewLog.html?buildId=69482&buildTypeId=bt431) outcome was **SUCCESS**
Summary: Tests passed: 1016, ignored: 12 Build time: 00:07:43


The Java Language Specification says that a new boolean array is initialized to all false, so this loop is redundant.


Perhaps this was changed recently, but it does not seem to calculate inches.  Seems to calculate wheel revolutions.


We don't need to update any grunt dependencies since those are being dropped.


@stefanpenner can you explain what you did here?


Well, that's a surprise :) I don't know how I could have helped you :)


Welcome @impronunciable!


Will implement.


I only spotted it because I was trying to work out if the priorities could be used in the hefce compliance checks - I think there's a slight disconnect here - we *should* use the published_online date for compliance measurement, but for RIOXX2 profile, published is the best fit!


@vilchik-elena ok, let's keep it as it is


This is throwing `Error: No redirect URI found(…)` @rogerhutchings


we stripped out the aadsso_ resets last night but this test slipped past (regression!).

Could you change 39 back to `reset_successful` and then change the reference on line 17?


@ralphtheninja I saw these are no longer used in node-gyp, and I made the same change in node-ninja so at some future date we can remove these lines of code IMO.


>  If a new autoloader is generated, why won't it work?

It deletes the files in the autoload.php file.


@stephanripke -- can you please add these fixes to your latest version (or accept the pull request)? I encountered this same problem when re-QCing PsychChip data this weekend.


Interesting, this is a file directly taken from the regular CC plugin since  I don't want to reinvent the wheel. Now is the question, do we keep this in sync or do we start modifying it ? Or maybe this class be upgraded to be part of C::B itself ?


Oh yeah, I also added a few constants which I forgot to commit separately :/


Thanks for the catch :-)
It's already [fixed](https://github.com/knutwalker/typed-actors/blob/65bd6ed11e9e9d2160006d32dbca90e55a167775/tests/src/test/scala/de/knutwalker/akka/typed/CreateInbox.scala#L34) though and in that particular case, `system eq sys` is true as well.


Add me as a friend, I'll add you to the group message.

https://m.facebook.com/profile.php?id=1271278123&tsid=0.14879214158281684&source=typeahead

On Monday, February 22, 2016, mariahcann1995 <notifications@github.com>
wrote:

> what's your Facebook name
>
> Sent from Outlook Mobile<https://aka.ms/qtex0l>
>
>
>
>
> On Mon, Feb 22, 2016 at 7:23 AM -0800, "jaybustah" <
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');><mailto:
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>> wrote:
>
> That would work better for me. Can you send a link?
>
> > On Feb 22, 2016, at 9:50 AM, mariahcann1995 <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> >
> > We're on a Facebook group now
> >
> > Sent from Outlook Mobile<https://aka.ms/qtex0l>
> >
> >
> >
> >
> > On Mon, Feb 22, 2016 at 6:06 AM -0800, "tweekley49" <
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');><mailto:
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>> wrote:
> >
> > I just added you. I'll add you to group when you accept it.
> >
> > Sent from my iPhone
> >
> > > On Feb 22, 2016, at 8:26 AM, mariahcann1995 <notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> > >
> > > I did I haven't gotten anything
> > >
> > > Sent from Outlook Mobile<https://aka.ms/qtex0l>
> > >
> > >
> > >
> > >
> > > On Mon, Feb 22, 2016 at 5:25 AM -0800, "tweekley49" <
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');><mailto:
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>>> wrote:
> > >
> > > Mariah, Adam added you on skype. Any chance you can accept it so we
> can get you in the group chat?
> > >
> > > Sent from my iPhone
> > >
> > > > On Feb 22, 2016, at 8:11 AM, mariahcann1995 <
> notifications@github.com
> <javascript:_e(%7B%7D,'cvml','notifications@github.com');>> wrote:
> > > >
> > > > Hey guys it is due tomorrow night at 2359....
> https://wiki.archivematica.org/File_Browser_Requirements look at this.
> > > >
> > > > -
> > > > Reply to this email directly or view it on GitHub.
> > > >
> > >
> > > -
> > > Reply to this email directly or view it on GitHub<
> https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16239951
> >.
> > > -
> > > Reply to this email directly or view it on GitHub.
> > >
> >
> > -
> > Reply to this email directly or view it on GitHub<
> https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16240940
> >.
> > -
> > Reply to this email directly or view it on GitHub.
> >
>
> -
> Reply to this email directly or view it on GitHub<
> https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16243034
> >.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/csu-cs/csci-325-file-browser/commit/ac3f88e378969a8996b65fc352f6f89471c6132b#commitcomment-16244122>
> .
>


This works accidentally, but URI escaping the *whole* URL seems a bit error-prone. I'd either just escape the param values, or (preferably) see if HTTParty will accept a hash and escape the param values for us.


Well, with different things: Ivy/Counsel, the workflow you put on Emacs.SE regarding Ag+Multiple Cursors. All things that really improved my daily workflow. :-)


Did you add a new source file to handle the multiple cameras?  If so, I do not see it here.


Have you signed in with it? If not, it won't have a redirect URI set


Da dove cominciamo maestro?

Mi ha contattato Bonnie Taylor... e gli ho parlato di te... Magari troviamo dei punti di collaborazione comuni?

 really like how your vision statement (in the “About” space) puts the global collective and collaborative emphasis on what you want to achieve in your project. Of course your presentation medium is very attractive too.

I have a similar vision for my proposed project, but my intended user base is more limited: medical students in the United States and Canada. I would like my proposal to have some of the forward thinking you express, but when I attempt to explain the significance of introducing this type of educational technology, it seems too grandiose.

Vi metterò in contatto :)


Fix Zendesk Async tasks issue


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16651](https://teamcity.mesosphere.io/viewLog.html?buildId=141136&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Error while applying patch Build time: 00:00:21


Glad that was useful :)


Good point, ACK


I think the easiest way is to just create a new commit to this branch.


You are correct @dkastl! I have commited it again.


VOXXY IS OUT OF MINDS, YAYA! IS GOODS YAYAYAYAYAYAYAYAYAYAYA...


Great!


No need for a dummy class here. Just use something like `"invalid"` or `0` to trigger a mismatched type comparison.


utilise la fonction is_horizontal(p)


Awesome!!! I love that you used a scope for this. :100:


Getting isn't appropriate in this method; you ought to be putting here.


Не полностью выполнено задание 1.
Не добавлены файлы status.txt и log.txt.
По именам коммитов не ясно что там происходит (более того они дублируются).
Задание принято, но можно доработать.


Separa el codigo en 2 métodos, mira el de postgreSQL https://github.com/demonio/ActiveRecord/blob/03d0656d4b3589ece5644f7b84578e145c20acff/lib/Kumbia/ActiveRecord/Metadata/PgsqlMetadata.php

Además, el código responde a tus preguntas


E.g., `self.assertFalse(name_val == "invalid")`


How many third party mixins do you typically use on a project? I'm onto PostCSS nowadays so a lot of stuff I need done is done behind the scenes (like Autoprefixer). But other than those things the only style "mixins" I use is a grid.

> The new axis will just be a collection of individual post-css plugins
Cool! I'll definitely check them out when you guys do it.


ok I understand the issue now. Maybe HqlQueryExecutor is initialized too early? Its name seems to imply it requires Hibernate to be initialized. Haven't digged either to know why it's initialized early.


This looks like you are assuming cheating happened if the device is ever rotated, independent of whether in fact the button had been clicked prior to the rotation.


This must be caused by GitHub repackaging the file - which is a little scary because they could insert anything here. But I'm fine with the update.


Never mind, I see, you rerouted it on the LCD side and the code by using pins already on the LCD header but not used.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16649](https://teamcity.mesosphere.io/viewLog.html?buildId=141134&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:04:16


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `tests/yaml/config.py`, line 1.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/yaml/config.py
+++ b/tests/yaml/config.py
@@ -1,4 +1,5 @@
-extends: default
+extends:
+    default

 rules:
   # 80 chars should be enough, but don't fail if a line is longer
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `tests/yaml/config.py`, lines 4 to 10.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/yaml/config.py
+++ b/tests/yaml/config.py
@@ -1,13 +1,12 @@
 extends: default

 rules:
-  # 80 chars should be enough, but don't fail if a line is longer
-  line-length:
-    max: 80
-    level: warning
-
-  colons:
-      max-spaces-after: -1
+    # 80 chars should be enough, but don't fail if a line is longer
+    line-length:
+        max:
+            80
+        level:
+            warning

   # don't bother me with this rule
   indentation: disable
```


**The code does not comply to PEP8.**
*PEP8Bear, severity 1, section `autopep8`.* *Related to file `tests/yaml/config.py`, lines 12 to 13.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/yaml/config.py
+++ b/tests/yaml/config.py
@@ -9,5 +9,10 @@
   colons:
       max-spaces-after: -1

-  # don't bother me with this rule
-  indentation: disable
+    colons:
+        max-spaces-after:
+            -1
+
+    # don't bother me with this rule
+    indentation:
+        disable
```


**Line is longer than allowed. (81 > 80)**
*LineLengthBear, severity 1, section `linelength`.* *Related to file `tests/yaml/YAMLLintBearTest.py`, line 27 column 81 to line 27 column 82*.


**Line contains following spacing inconsistencies:
- Trailing whitespaces.**
*SpaceConsistencyBear, severity 1, section `python`.* *Related to file `tests/yaml/config.py`, line 8.*

The issue can be fixed by applying the following patch:

```diff
--- a/tests/yaml/config.py
+++ b/tests/yaml/config.py
@@ -5,7 +5,7 @@
   line-length:
     max: 80
     level: warning
-
+
   colons:
       max-spaces-after: -1

```


I meant 5.3


The following doc(s) have been deployed to the [cilogs](http://cilogs-static-docs.pantheon.io/docs) Multidev environment:
- [scripts/deploy-live.sh](https://github.com/pantheon-systems/documentation/commit/0975342b7106417e4fea4c04a7906e5c868e3a8d/scripts/deploy-live.sh)
- [scripts/deploy-multidev.sh](https://github.com/pantheon-systems/documentation/commit/0975342b7106417e4fea4c04a7906e5c868e3a8d/scripts/deploy-multidev.sh)


I think I'd prefer to just see the name value here. `__repr__` can stay the same, keeping the full description of the object, but `__str__` should hide those details.


The checksum changed because GitHub must have repackaged the file. Fine with me although I realise that this is a potential security problem.


Also note that although you have the `savedInstanceState` influencing (at least through its existence) the result, you don't have it influencing the display, so rotating the device after pressing the button makes the divulged answer go away.


we don't test mainframe for opensuse, not ? :)


I needed to fix a number of issues with the SpeechService and also add some more features to the configuration so created a new branch.  I merged this PR in to the features/VoiceConfiguration branch, please could you take a look at that and see if I've mucked anything up or if the merge is good?


Awesome, thanks :)


please update to transiflex


in the other example, this is in the `if __name__ == "__main__":` block


`__repr__` looks good but I think `__str__` should just return the string value, not the full representation.


Я русский тоже сделаю потом =)


The charts no longer extend JFrame, so applicationTitle is not necesarry. More information here: 6056286592ecc7ec46f35ecd51345db61e3367f4


We use optional dependencies for features that are disabled by default (e.g. compression, percentiles). Also, they require an extra configuration step, which makes things a bit more complicated.

Here I think users would prefer using the pid given the choice, and we have a solution that should work reasonably well on all platforms. So I would not make the dependencies optional. If this is a problem for C* or DSE, they can always exclude the dependency and it will fallback to other methods.


.gitignore надо бы выносить в отдельный коммит с соответствующим названием


about the qor.task??? can you make it into .md... I don't know... it's an audience thing... XD


just write
```perl
if (snapper_is_applicable && get_var("BOOT_TO_SNAPSHOT")) {
```
without the parentheses


This means that even if they cheated on the question before, if they come back around to it you are wiping the slate clean and saying they haven't cheated on it.


Electric-Service-Refactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (2147 of 2151 tests passed, 549 test warnings)

 * 553 tests had: AUD diffs.
 * 483 tests had: RDD diffs.
 * 1 test had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


This call is not save.
Add a nullptr check to the music type detection of BGM_Stop()


Should we give the repo a nice friendly URL ?
ex. maven.affectiva.com


LGTM


I've yet to make the post-scss plunge:

i've got it down to
```
  "dependencies": {
    "knapsack": "^0.2.4",
    "include-media": "^1.4.1",
    "jeet": "^6.1.2"
  }
```

you have gchat? I'm danielbox@gmail.com


wcięcia (4 spacje?)


удалить все коммиты и залить с начала?

22 февраля 2016 г., 19:08 пользователь Tatiyana Domanova <
notifications@github.com> написал:

> .gitignore надо бы выносить в отдельный коммит с соответствующим названием
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EugeneSayko/webChat/commit/5068ac3a9fe26264b7741ddcdcb1c60d78f225d8#commitcomment-16244437>
> .
>


I don't understand why you need to change this line. Didn't tidy catch this in before?


unless you object I would like to keep helper methods in the support directory, we have a file `support/helper_functions.rb` that should auto-include it if you paste it in there.


Electric-Service-Refactor (Myoldmopar) - x86_64-Linux-Ubuntu-14.04-custom_check: OK (0 of 0 tests passed, 0 test warnings)

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


Задание 2.1 принято.
Постарайся лучше называть коммиты (избегать дубликатов и делать так чтобы было ясно что изменилось в проекте)


niepotrzebnie całość tutaj wciąłeś


That's even better, great! :+1:


porque hay que agregar la gema "daemons"??


@pynicolas From first sight it seems that what is returned is the result of `createSimpleBlock(successor)` call, but `build(subFlowTree)` might reassign `currentBlock`, is that wright?


This might need to be in actual project folder root to work right. My editor at least seemed to ignore it, even with the editorConfig extension installed. :shrug: I'll keep playing with it.


if tender.awards and ...


I did not used RADDS LCD connector - I do not have any - I build my own based on wiring described on Readme


It is copied from line 182, perhaps could need the same check there?


`# Draft service lifecycle events`


well, what is the meaning of this line then?

`path := Concatenation("C:",...`

Intentional code obfuscation ? :-)

On Mon, Feb 22, 2016 at 2:34 PM, Christopher Jefferson
<notifications@github.com> wrote:
> @alex-konovalov : You are right. That's a weird way to write things, but is
> correct :)
>
> —
> Reply to this email directly or view it on GitHub.


LGTM


do nothing here, is not supported at all


> Maybe HqlQueryExecutor is initialized too early?

That's for sure since it's totally useless without the hibernate but it's not like it was forced to be initialized when application start :) It's probably just that it's indirectly injected by some entry point.


niepotrzebnie wciąłeś


Стили у кнопки по ховеру. не должно быть тени сверху.


The following doc(s) have been deployed to the [1366ghtem](http://1366ghtem-static-docs.pantheon.io/docs) Multidev environment:
- [/ISSUE_TEMPLATE](http://1366ghtem-static-docs.pantheon.io//ISSUE_TEMPLATE)
- [/PULL_REQUEST_TEMPLATE](http://1366ghtem-static-docs.pantheon.io//PULL_REQUEST_TEMPLATE)


In `onSaveInstanceState` you use `putBooleanArray`, yet here you use `getBoolean`. This mismatch is a sign of trouble. Relatedly, you wind up setting all the elements of `mIsCheater` to the same value, so the user will either be given a judgmental response on all questions or on none of them, rather than some mix that reflects their past cheating.


typo ?


хорошо, больше не повториться

22 февраля 2016 г., 19:09 пользователь Tatiyana Domanova <
notifications@github.com> написал:

> Задание 2.1 принято.
> Постарайся лучше называть коммиты (избегать дубликатов и делать так чтобы
> было ясно что изменилось в проекте)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/EugeneSayko/webChat/commit/6e53035a8b76b5f0422faf7b2d4061ce0c4fbd33#commitcomment-16244488>
> .
>


Almost LGTM


Right, we had a lot of problems like this one because we kind-of assumed a coarse-grained locking tree from the very beginning. They are being fixed in the concurrent version, where every underlying tree operation returns all relevant versions.


@thebigsmileXD Thank you very much!


LGTM!


I'm not sure, I wouldn't think so. @nouiz what do you think?


Недочеты:
 - нет изменения status.txt из клонированного репозитория ("Копирование удаленного репозитория и работа с ним" пункт 4), должен был бы быть отдельный коммит.

Задание 1 принято.


а якщо створювати одразу в active стані?


Yeah but here bgm is != nullptr (see line 180).
Either add a nullptr check in the BGM_Stop function or replace your BGM_Stop() call with "bgm_stop = true"


K.


It looks like this [line](https://github.com/alphagov/govspeak/blob/384a0958c86c7b865787ab715f2542ae08f79188/lib/kramdown/parser/kramdown_with_automatic_external_links.rb#L12) throws some light.. I have changed the markdown link to \[certificate of no impediment (CNI)](//www.gov.uk/government/publications/example-certificate-of-no-impediment-cni) and it generates ahrefs without the external tag on it..


sprawdź więcej cech

np wiesz, że każde wideo na yt musi mieć tytuł => czyli możesz sprawdzić, że tytuł nie jest pusty => a jakby był, to podpowiada, że gdzieś parser ma buga (albo yt zmienił htmle)


e la madona (alla Pozzetto)


:waiting:


good catch thanks!


why is this a `.py` file?


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 16654](https://teamcity.mesosphere.io/viewLog.html?buildId=141142&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1, ignored: 234 Build time: 00:04:40


nice ci broken af, could have been coded by leet


LGTM


hm, I think it'd be nice to autogenerate the config, Lint provides facilities to do that.


Download = загрузка, не применимо к файлу.


This is because `enrich_ip` is a string in a file. There are better ways to handle it but this seems like Alex is applying a quick fix, we can re-address later of course :)


Aside from order, lines 63-68 duplicate lines 52-57. They should be factored out.


У активной ссылки нет подчеркивания


Dodane w commicie

https://github.com/Glenpl/LYTM/commit/7ec65e85ff5ceb0069fb1896f1a3dbe3ed59ee69


откуда пользователь узнает как называется файл?


@pynicolas Looks like we need one interface for `ForInStatementTree` and `ForOfStatementTree`.


fixed!


h3h3 ist schwul - wie ein schwuler


Electric-Service-Refactor (Myoldmopar) - x86_64-MacOS-10.9-clang: OK (2109 of 2113 tests passed, 546 test warnings)

 * 550 tests had: AUD diffs.
 * 480 tests had: RDD diffs.
 * 1 test had: ESO small diffs.
 * 7 tests had: MDD diffs.
 * 7 tests had: MTD diffs.
 * 2 tests had: ERR diffs.
 * 4 tests had: ESO big diffs.
 * 1 test had: MTR big diffs.
 * 4 tests had: Table big diffs.
 * 1 test had: EIO diffs.
 * 1 test had: SSZ big diffs.
 * 1 test had: ZSZ big diffs.

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


This slipped through because a regex pattern was too flexible, which should be fixed now.


inline


В корзине есть товары. Укажи в соответствии с макетом.


sprawdź dla każdego znalezionego


Удали закоментированное.


przekracza maksymalną długość linijki (80 znaków)


@dimpase: no, it is called ["look out for separable and non-critical subfeatures when working on something"](http://manishearth.github.io/blog/2016/01/03/making-your-open-source-project-newcomer-friendly/). Anybody who wants to practice in issues and pull requests is welcome to make the code less obfuscated.


Shame!


tutaj dane masz „predefiniowane” – sprawdź realne dane


@shaunanoordin Rog has been suggesting to put all the explicit bindings in the constructor. E.g. https://github.com/zooniverse/wildcam-gorongosa-education/blob/master/src/containers/Classrooms.jsx#L12
It's clearer to read.


@qpeyramale Thanks for reporting. it has been fixed by 01f2046e23543cf15525f3b21e31c1d69030068f


TeamCity ClearMeasureBootcamp :: 1 - Integration Build [Build 1.2.29](http://build.clear-measure.com/viewLog.html?buildId=5067&buildTypeId=ClearMeasureBootcamp_1IntegrationBuild&guest=1) is now running


Diving in to a failing example context, we can see that the events returned by the association are out of order

```
[34](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> message.info_request_events.map { |event| [event.id, event.params[:smtp_message_id]] }.compact
=> [[936, "ogm-27+56cb2e5560370-93e0@localhost"], [937, "ogm-27+56cb2e557e4d6-f459@localhost"], [935, "ogm-14+537f69734b97c-1ebd@localhost"]]
```

We can make them come back in the expected order by forcefully ordering by `id` or `created_at`:

```
[35](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> message.info_request_events.order('id DESC').map(&:id)
=> [937, 936, 935]

[36](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> message.info_request_events.order('created_at DESC').map(&:id)
=> [937, 936, 935]
```

Ordering by `created_at` is a bit bizarre, since they all have the same `created_at` timestamp:

```
[47](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> message.info_request_events.map { |e| e.created_at.to_i }
=> [1456156245, 1456156245, 1456156245]
[48](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> message.info_request_events.map { |e| e.created_at.to_i }.uniq
=> [1456156245]
```

Even if we unscope the query, and rescope by `outgoing_message_id` the order is incorrect.

```
[33](#<RSpec::ExampleGroups::OutgoingMessage::SmtpMessageIds::AResentMessage>)> message.info_request_events.unscoped.where(:outgoing_message_id => message.id).map(&:id)
=> [936, 937, 935]
```


Неправильно понял... В следующем коммите поставил все скрипты в конец.


не стоит надеяться на перевод строки. Gson позволяет читать массивы и коллекции (gson.fromJson(reader, Message[].class))


TeamCity Chorus :: chorus-win32-master Continuous [Build 2.6.185.8515507208c1539220ebee854cf5e7e479e1de6d](http://build.palaso.org/viewLog.html?buildId=69480&buildTypeId=bt2) outcome was **SUCCESS**
Summary: Tests passed: 943, ignored: 50 Build time: 00:38:37


@caseyedavis12 : I don't really understand how this merge worked, so please check that the content is what you want.


ok,
did accept the pull request.
is that enough or do you need a new complete version?

On Mon, Feb 22, 2016 at 5:00 PM, jigold <notifications@github.com> wrote:

> @stephanripke <https://github.com/stephanripke> -- can you please add
> these fixes to your latest version (or accept the pull request)? I
> encountered this same problem when re-QCing PsychChip data this weekend.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/Nealelab/ricopili/commit/2cb6d4a1e991fcad51fd9974694c2997ce0c599e#commitcomment-16244200>
> .
>

--
Stephan Ripke, M.D., Ph.D.
Analytic and Translational Genetics Unit
MGH Simches Research Center
185 Cambridge Street - CPZN6818
Boston, MA 02114
USA

sripke@broadinstitute.org
ripke@atgu.mgh.harvard.edu


add minecarts pls


то же самое - можно сохранят коллекцию.


should this include .webvtt ? @egreaves


Размер селекта не верный. Стандартный треугольник скрой, сделай через background.
А также неправильно выравнены элементы.(отступы)


ist das jetzt anstelle des Cronjobs?