@yyuu do you feel comfortable with the changes in #549 yet? I haven't encountered any issues with it so far. If you think it's good to go, I can take care of pushing a new release up to homebrew.


nope. directly copied it.

and the button code itself works because i directly copied it from my rockpaperscissors project, just replaced the function name, and it works there. [look for yourself](bergenfrundt.github.io/rockpaperscissors)


fast-forwarding master to auto = 84130a9c


TeamCity Device Info :: Tests [Build 52](http://82.35.147.206:8111/viewLog.html?buildId=238&buildTypeId=DI_Test) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:01:21


The error would it isn't able to unmount because there's still references.


Good work on escaping those pesky danish letters.

Keep it up!


This function would probably called without source when the Pokemon faint from things other than direct damage, i.e. recoil, weather, status, etc.


the fuck


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


Might be worth using a more efficient algorithm eventually, but it's not worth it now.  Could be worth a TODO, though.


Okay, I wasn't sure exactly what was null. How is it even possible that the event handler is null? I'm not familiar with enough design patterns to know if a default event handler would be acceptable here.


The fix is simple I need a condition surrounding the save.
The fact that I am returning the ID is necessary in order to save it within the contains table.


I'm never sure what direction people mean when they refer to "leaves" in a dependency context, and I'm pretty sure I've heard people mean both ways. "Sinks" and "sources" are good.


TeamCity jonas-requests :: WindowsBuild [Build 33](https://build.neohq.net/viewLog.html?buildId=574404&buildTypeId=JonasHaRequests_WindowsBuild) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 28001, ignored: 97 Build time: 1:33:28
### Failed tests
```
org.neo4j.test.ha.ClusterTest.testClusterWithHostnames: <no details avaliable>

```


Separate Management Boards


:thumbsup:


Yay~!


Maybe that's not the problem? I see that CouchDB documentation sugggests to use SSL to enhance the security problem, and when in production mode, we always use https scheme to connect to CouchDB, I think if this is the problem, then a server created as "server = Server('https://root:passwd@127.0.0.1:5984')" will also be a problem since no protect about the root's password here.


yes


I changed it back to your repository.


Nice job, finishing the project, guys!


@yyenigun thanks for your reply. I created a pull request. I send the pull request to "develop" but I created the branch from"master". I hope this wont cause any problem.


I also haven't encountered any issue after #549. Please go ahead. Thanks for taking care! :octocat:


There will always be one cookie here if authorized, according to docs


i will merge the PR for internal testing firstly.


ok, I will add spaces


ok, I will use '_' to ignore second and third value, and your suggestion is much nicer


yes, change the long side into vertical style


yay for your first commit @cchun8! huzzah! /cc @figureone ;)


sed is not necessary. It's better to keep the timezone in gnuplot datafile and instruct gnuplot to ignore it.


You need a better commit message. Why was this change necessary? What does it fix?


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


This commit caused #8 !


Everything seems good! :+1:


No need to check if err == nil.


// res and err here are directly from


allow me to leave a comment next to a creepy icon of my face on one of your commits


nice getting rid of spaces


#5507-Polygonal-and-TriangularDoor-have-the-same-value (mjwitte) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1595 of 1595 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)


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


This was on my personal todo list, thanks for taking care of it!


Nice. GL at LTC!


#5507-Polygonal-and-TriangularDoor-have-the-same-value (mjwitte) - 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)


I'm not seeing those triangles in Windows


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-UnitTestsCoverage-Debug: OK (1022 of 1022 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-30.59%25-green.svg)


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - 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)


yes, it does. disregard that comment.


To ensure there are clinical reminders, just create a patient with age less than 5 years old. Then a bunch of them should show up there.


:100:  屌屌的.


Regarding the batchcom blue line, yes, just remove it.


fast-forwarding master to auto = c4cfc495


Ok


Make sure you configure your editor to use a new line at EOF. Git usually warns you about that. See the warning in the UI above.

Also, use consistent 2-space indentation. and do not leave white-spaces (/\s+$/). You can configure your editor to automate this for you.


@bpowell Ok this breaks, it should FAIL because size/=1 but it wont compile... any tips?


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


Why not store the objects in the iSelector?

What if I do this:

"some text" ⇥ append to... ⇥ "File that is deep in some folder"

I'd kinda want that file in my history


I've never used Git rebase before so I'm not completely sure if this is correct. I did git rebase -i HEAD~10 on my branch and pushed the changes to my branch again.


I'd still like you to clarify this and explain why :)


Also references tomlongo/nationbuilder-ios@b488e8f


:+1:

I do not know how backwards compat this is tho...

Maybe it's time for 2.0.0 ? Make it min="2.5.x" max="2.x.x" :)


Add reqfile.txt in the project directory and ignore it via .gitignore files.


I'm wondering what other information is lost here when you do this. What's in the `historyObject` `NSDictionary`? Is there anything useful other than what's in the `selection` key?


@montefra I've submitted #119 per your request.


On the [C# version](https://github.com/QuantConnect/Lean/blob/master/Algorithm.CSharp/LimitFillRegressionAlgorithm.cs#L46), we compare the System.DateTime.DateTime at System.DateTime.Ticks resolution which is higher than microsecond. In this case we may not need it, but I want to be as close as possible the C# version.


If you're rebasing, then it might make sense to remove this commit and the commit it reverts - to keep things tidy?

Although it'd mean all my nice comments on these commits will be lost! :(


This line is too long. Break the words like this:

```
Bad:
d = {key1, value1, key2: value2, ...}

Good:
d = {
    key1: value1,
    key2: value2,
    ...
}
```


2098-airloop-diagram (kbenne) - Win64-Windows-7-VisualStudio-12: OK (2372 of 2397 tests passed, 0 test warnings)

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


Hm. I thought `.zshrc` was _only_ [supposed to be sourced for interactive shells](http://zsh.sourceforge.net/Intro/intro_3.html). Also, that page says (of `.zshenv`/`.zshprofile`):

> as a general rule, it should not change the shell environment at all

Confusing, because I think OS X does some weird things here. As far as I can tell, it runs `.zshenv`/`.zprofile`/`.zshrc` (in that order) for new terminal windows, and `.zshenv`/`.zshrc` if you invoke a subshell with `zsh`. If you run a Zsh script that has a shebang line, it runs `.zshenv` only. Don't have access to a Linux box right now, so I can't test it, but I bet it does something different.


I tried this too. is this work for all terminal?


1. Change .gitignore file, removing .idea

2. Change function names and coding styles to make it consistent with other parts


be more specific with commits - i.e. `script now does _____, no response error msg`


@cchun8 nice job! Your new post can be viewed on the live site [here](https://assist.coe.hawaii.edu/how-to/2016/03/08/bbc-mac-launcher.html).


@benjamn I am interested in your thoughts on this stubbing technique.

 - Is this the best way to stub an import? (the goal here being to change the definition of  `Todos` before the factory code runs).

One downside is that this forces you to use `require` --- although I think normally you'd want to put this code in a `beforeEach` block (and undo it in an `afterEach` block) anyway[1]

[1] Although does that make sense anyway? Is there a way to later import the factory and *not* have it depend on the mocked symbol (say in a different test file)


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


Hi Rod,
Separating these things out may  be tough to do. The UI developer also "cleaned" up the look in some places with minor changes (for example, here placed brackets around the label to make it consistent with the rest of the links on this page). There are also several changes in the code to support the ui pills better, which goes hand in hand with the theme change.


Felix, how does this code works when the store is missing only in few nodes, but not the rest.


What do you mean? I did invoke it.
(*list)->element.vt->release(&(*list)->element);
currently, only triangle_mesh's release function is not empty.


My guess is that the majority of motions don't make sense in multiple windows, so having a global toggle isn't very useful. I'd rather the macro option be the only way to enable it. (And have it default to off.)


I would say
whether or not add new store, if the store is not found in the cluster.


so this scares me a little bit, since if you go up through the code, `splitObjects` *could* return a mutable array.

`splitObjects` come from `[object objectForCache:kQSObjectComponents]`.
`setObject:objects forCache:kQSObjectComponents` is set in `-[QSObject objectByMergingObjects:(NSArray *)objects]`
But... sometimes a mutable array is passed to `objectByMergingObjects`. Take for example [here](https://github.com/quicksilver/Quicksilver/blob/master/Quicksilver/Code-QuickStepCore/QSObject_FileHandling.m#L620).
`-[QSObject fileObjectsWithPathArray:paths]` returns a mutable array.

I'm not saying this is a problem with your code, but we need to be aware that `splitObjects` might be a mutable array at some point.
My suggestion is that in `-[QSObject objectByMergingObjects:(NSArray *)objects]` we just do this instead:

```
[object setObject:[objects copy] forCache:kQSObjectComponents];
```

but I'm unsure of the implications of this. Should be fine I think. We'd need to check if `-[QSObject splitObjects]` is ever mutated at any point, but my guess is the answer is 'NO' since it's marked as returning an `NSArray` and not an `NSMutableArray` anyway


enviar a cuenta allied wallet


Why not use a static var since it never change.


Hi Zenon, sorry, overlooked the line note and could not see a message here corresponding to the email notification I received.

I'll look into this again on Monday.  So far, I can see accounts.User._meta.swapped should be set as well (confirmed by pointing AUTH_USER_MODEL to auth.User) and the allow_migrate_model() method should see that.

I'll check whether the accounts AlterField migrations really stay active even when accounts.User is swapped out.

Cheers,
Vlad


I'll look into these...


If there's no global option, then there's no reason to use `cl-defun`, right?

I'm not against using it, but I don't think it's needed right now.


Thanks @billymeinke and @enriquezm! So much fun and great teachers @parkjc @fjumawan


Sure


I think we need a macro for this. Something like:

`#define CONTAINER_TUPLE(a, b) (a * 4 + b)`


...looking at `splitObjects` it could also come from `[object children];` and a quick scan of the code shows that we're setting mutable arrays for `children` in some cases. Maybe `-[QSObject setChildren:]` should also be changed in the same manner.

And why is setChildren: doing this:

```
        if ([[self cache] objectForKey:kQSObjectChildren] != newChildren) {
            [[self cache] setObject:newChildren forKey:kQSObjectChildren];
        }
```

and not this:

```
        if (![[[self cache] objectForKey:kQSObjectChildren] isEqual:newChildren]) {
            [[self cache] setObject:newChildren forKey:kQSObjectChildren];
        }
```


I agree this is the most useful option of your 3 options.


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


#5507-Polygonal-and-TriangularDoor-have-the-same-value (rraustad) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: OK (1580 of 1580 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-70.87%25-green.svg)


The problem is that I need to trust you in order to believe that you actually run that code on the server side, and you can hotfix it without anybody knowing if you are ordered to.
No offence intended, but on the internet by default you should not trust anyone.
Yes, prep your tinfoil hats.


Haha, my whole whine about `splitObjects` perhaps being mutable is blown away by this change.
Still, it's possibly something we need to fix, right?


Try clicking the smiley besides the icon, you should see it.


ooh nice!


True - I would have done that but the convention with the Builders is to use "thatIsNotDeleted" and "thatIsDeleted". The intention is that the Builders provide us with easy to read unit test code, so I think the two methods should remain.


Пожалуйста, не ставь эмоджи в названиях коммита.
Во-первых, картинки отвлекают
во-вторых, в консоли вместо картинки непонятное слово
![screen shot 2016-03-11 at 12 59 10](https://cloud.githubusercontent.com/assets/223620/13699078/269b4582-e789-11e5-9883-6faeba93ddb8.png)


Is that only a memory error?
If we can't allocate memory now, shouldn't we die? we probably won't be able to allocate in the future either


No changes are made mate.


哦哦  好哒，我放在上面只是说明一下用了什么歌而已- -


TeamCity Civic Assembly :: Build all [Build 93](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1838&buildTypeId=CivicAssembly_BuildAll&guest=1) is now running


5073_GroundDomainErrorMessage (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2132 of 2135 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-99.86%25-yellow.svg)


Thanks.  Will post another round shortly. Hopefully I found all comments.


What is the purpose of this change and why don't you make PR?


It would be better to deal with overriding column nullability in `ModelBinder#handleNaturalIdBinding`, but a change would have to be made to `SingularAttributeSource` API  to find out if not-null was explicitly mapped.


Kleine opmerking: `$ln` is weinig beschrijvend als naam van een variabele. Beter `$language` gebruiken. Voor de rest ok.


TeamCity Todo :: Build [Build 61](http://109.103.226.38:8512/viewLog.html?buildId=38288&buildTypeId=Todo_Build) is now running


@bcoca @resmo @robynbergeron @gregdek @jimi-c
Not sure if this is the way to comment on a proposal.
I doubt this is visible beyond the person who made the commit.


TeamCity Todo :: Build [Build 61](http://109.103.226.38:8512/viewLog.html?buildId=38288&buildTypeId=Todo_Build) outcome was **FAILURE**
Summary: Tests failed: 4 (4 new), passed: 2 Build time: 00:00:11
### Failed tests
```
adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): our todoList should be empty at first render: <no details avaliable>

adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): should add it: <no details avaliable>

adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): for empty todo it shouldn't add it: <no details avaliable>

adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): for todo with full of white spaces, shouldn't add it: <no details avaliable>

```


@AByzhynar changed in https://github.com/AMikhniev/sdl_atf/commit/b33625c2f6620856a088985120a34c397347a4d6


This is all fixed in the latest codebase


I'm going to try to revert the fix and see what happens.


TeamCity Todo :: Build [Build 62](http://109.103.226.38:8512/viewLog.html?buildId=38289&buildTypeId=Todo_Build) is now running


TeamCity Todo :: Build [Build 62](http://109.103.226.38:8512/viewLog.html?buildId=38289&buildTypeId=Todo_Build) outcome was **FAILURE**
Summary: Tests failed: 3, passed: 3 Build time: 00:00:06
### Failed tests
```
adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): should add it: <no details avaliable>

adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): for empty todo it shouldn't add it: <no details avaliable>

adding new todo.PhantomJS 2.1.1 (Linux 0.0.0): for todo with full of white spaces, shouldn't add it: <no details avaliable>

```


taupage init won't start because of it, and you'll have more tests failing (that's what we discovered yesterday, hence this fix)


Sorry, I don't understand what you are asking.

> On 11 Mar 2016, at 09:48, abukabhs <notifications@github.com> wrote:
>
> Do you have a ready made template that one can copy, upload to a server and slightly modify it with his domain, DNS ip and ISP IP ?
>
> Regards,
>
>
> > On 11 Mar 2016, at 01:18, Anton Belodedenko <notifications@github.com> wrote:
> >
> > Your guess is as good as anyone else's.. Probably.
> >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


I would suggest to move these issues to correspondent groups. There're issues about standard library, debugger, our gradle build plugin. This way it may seem that those are all compiler improvements.


Just thinking: shouldn't we ensure that the buffer is terminated with 0 ?


World.cpp - 695, 696 & 697 duplicate lines.


another option would have been to fix the grains to consistently use 11.x for sle11.


Yeah, this seems wrong.
Do you wanna send a PR fixing it?
Otherwise I will review it as soon as I have some time


This is what I meant, but it doesn't matter. Basically I've just noticed that we often do this:

```rust
if crate == LOCAL_CRATE {
    /* get information from tcx */
} else {
    /* get information from cstore */
}
```

It'd be nice to either move the information into cstore OR add accessors on the tcx that encapsulate the `if`. But this is not even a *nit*, just a passing comment.


Let's not forget paches


I just marked it for 1.1.0.RC1 (but I think it was probably in M5).


jut!


This is missing a free(otp) I think (was already missing before ...)
Same for the following ones


@thet Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plone.app.z3cform/15/violations
```
plone/app/z3cform/inline_validation.py:15:1: C901 'InlineValidationView.__call__' is too complex (15)
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


this is doing nothing. Probably should remove it


Can you say why the `.iml` files were modified in this commit?


Set在IE下貌似是不兼容的，试试看。如不能，参考image过滤那的去重处理（lodash）


this commit lgtm


in that case you may not need the `@` thing, only `./src/assets/fonts/**/*`


鸡兔说明


activated ?


完善


The sync invocations in the maintainer scripts should be pretty much superfluous, given that a disk error/outage while running these scripts would leave the system in an unclean apt state anyway. Remove them?


I look forward to your reaction there, but I'd like to go ahead and state that I strongly disagree with you. The XML should (within reason) represent all details from the original binary. The compiler is free to apply some optimizations (I don't see that as "disobeying" at all, btw.), but it is crucial that we can see which things it chose to change. TTX output is also meant for debugging and analysis. Sure, there can be some surprises if a user doesn't (yet) know how this works, but to HIDE such optimizations from the TTX output is fundamentally wrong is it makes it impossible to see how "truthful" a round-trip was, and can lead to much greater surprises (as evidenced by the PUSHW/PUSHB vs. JMPR thing).


注释不用写这么麻烦吧
```
// 去重
```


Doesn't it make all the fonts copied to /dist/. ?


zdsvdvzvx


TeamCity Todo :: Build [Build 63](http://109.103.226.38:8512/viewLog.html?buildId=38291&buildTypeId=Todo_Build) is now running


Anti pattern in usage. Use the concrete exception


TeamCity Todo :: Build [Build 63](http://109.103.226.38:8512/viewLog.html?buildId=38291&buildTypeId=Todo_Build) outcome was **FAILURE**
Summary: Exit code 127 (new) Build time: 00:01:02


This deletes an old file


Would be a nice small improvement to check inside the loop if the extent really changed.


if you can use a config var instead of relatve directory


I think the differences in our views can be summarized as follows:
1. TTX is an instruction for how to compile a font (Behdad)
2. TTX is a truthful dump from an existing font (Just)
Of course it's a bit of both, but I think we're losing important functionality if we abstract away too much from 2.


Shortlog of HEAD commit is too long.

*GitCommitBear, severity NORMAL, section `commit`.*


Woocommerce can also be activated per site, so it should check both options.


I'm not fully understand what the problem? Does that ros implementation issue?


This change will bring a new notice, see attachment.
<img width="1181" alt="screen shot 2016-03-11 at 11 13 36 am" src="https://cloud.githubusercontent.com/assets/7510360/13699426/8bbfbd3a-e77a-11e5-9c00-83a6fb1b142b.png">


> I look forward to your reaction there, but I'd like to go ahead and state that I strongly disagree with you.

Just to be clear, this particular case, and the case of simple glyf-table glyphs flags, I didn't make any changes.  So from what I understand, you are disagreeing with your previous self as well :).  I just tried to harmonize how things are.  I'll explain my view of how I made the decision to change PUSHB/PUSHW and remove format in a couple of places in that bug. :)


> I think the differences in our views can be summarized as follows:
> 1. TTX is an instruction for how to compile a font (Behdad)
> 2. TTX is a truthful dump from an existing font (Just)
> Of course it's a bit of both, but I think we're losing important functionality if we abstract away too much from 2.

Can you copy this comment to https://github.com/behdad/fonttools/issues/92#issuecomment-65170183 so we continue there?


Where are the libsecret_p.h and libsecret.cpp?


My heuristic is:

- if the output is a part of the command's use case, it should use `fmt` - e.g: `status`
- if the output is informational, then it should go through the logger - e.g: error messages

I am open to changing this, but I like that we can change the output as needed.
Either way, I'm not entirely certain it's a major issue, as it may disappear when we go client-server - the client uses `fmt` on the whole, and the server logs everything.


Thanks for the FYI! I need to start relying less on Drone and doing more locally :)


@AByzhynar сhanged to be the same in https://github.com/AMikhniev/sdl_atf/commit/0349d2c53c44c2b185cdf5660dce9816a9ac5d88


@AByzhynar done in https://github.com/AMikhniev/sdl_atf/commit/0349d2c53c44c2b185cdf5660dce9816a9ac5d88


If do this in odoo role - is code duplication
You can simply indicate these variables in playbook.yml


I believe you should check for `this.isDestroying` also.


这周我们这市质检，考完数学之后觉得药丸啊
大题看着好熟悉，但就是不会做(╯‵□′)╯︵┻━┻
答题卡的大题几乎是空着的啊！！！
第一次考成这样(╯‵□′)╯︵┻━┻

-----

高二的话大概再一两个月就开始总复习了吧，到时候节奏就会开始抓紧了


console log driven development


... difficult. Here it's the fonts directory from the client side. So "../" + our "fonts" directory in "dist"


o


ah! I remember !

`@(fonts)` keep the folder in destination so yes it is required


first


Does it work?


I created a separate issue: https://github.com/c2corg/v6_api/issues/181

Btw. nice tour! Added to my todo list. :)


Will do that. Sorry, it's indeed off topic here, I didn't even look at the details for the components.flags issue...


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


This won't work on some Postgres installs including Turkey...see the comments on the deduplicate you've removed:
if current.deployment_settings.get_database_type() == "postgres":
 -            # Python lower() only works properly on Unicode strings not UTF-8 encoded strings
 -            # Oddity:
 -            # Python lower() converts the TR char İ to i (Correctly, according to http://www.fileformat.info/info/unicode/char/0130/index.htm)
 -            # PostgreSQL LOWER() on Windows doesn't convert it, although this seems to be a locale issue:
 -            # http://stackoverflow.com/questions/18507589/the-lower-function-on-international-characters-in-postgresql
 -            # Works fine on Debian servers if the locale is a .UTF-8 before the Postgres cluster is created
 -            query = (table.name.lower() == s3_unicode(name).lower().encode("utf8"))
 -        else:
 -            query = (table.name.lower() == name.lower())


@tgr-odoo


Uploaded!


```ruby
# @param resource [ActiveRecord::Base, Reform::Form]
        # @param attributes [(Hash)]
        # @return [ActiveRecord::Base, Reform::Form]
        def assign_attributes(resource, attributes)
          if resource.is_a?(::Reform::Form)
            case action_name
            when 'new', 'edit'
              resource.prepopulate!
            else
              resource.validate(*attributes)
            end
          else
            super(resource, attributes)
          end
        end
```


What is this function used for? Does the validateContact function not work as expected in DecoderUtils?


*init'd


left this softaculous url here by accident.  how do I delete it?


Thanks for renaming


Sounds fine, lets see what we get.


This looks a lot cleaner now :+1:


NB In general very happy to see all this work, thanks a lot :)
This edge case is a classic reason why centralising the function is so useful of course!


done :)


We should include this in RelationshipRecord as well


not initd, when you say it out loud it sounds like an init daemon lol


+1, would add a "Performance" subsection in each of the following sections


STF_EQUAL has still compilation problems


Would tripletData be better off being a Hashtable given that you need to iterate through it to get values?


I think front-end and back-end are rather technical terms with no definite meaning for an outsider, so we should not use them in the change log. I have no suggestions though


isn't it worth adding `QVERIFY( mCanvas->extent() != originalExtent );`here?


No, actually this is how it works for Turkey ;) it didn't work before.


Explicit scope. Should probably be private like the other member variables.


Can you update the logging titles:
```if (DEBUG) Log.d("getInsertIntent", "ContactTriplet has a home meta");```
Becomes:
```if (DEBUG) Log.d("addEmailDataToIntent", "You know better what to put here than what I have");```


Conversion to lower in multiple places is impacting the code readability and structure. The best way is to fetch the exact name from Azure using Get-AzureResourceGroupName(need to  handle utilities) before starting the actual functionality.


No benefit. I used it just because it was a little bit pain to find a way to avoid ambiguous method definition here. I'll resolve it later if possible.


Could we pass in the created_at if it needs to be set for a test with an unpersisted record?


I'd change "front-end" -> "analysis & diagnostics", "JVM back-end" -> "JVM codegen", "JS back-end" -> "JS translator", but this is only my personal opinion


done


Is there a reason these are not listed in the same way as the previous issues, in a list with the explanation for each one?


A li closed without an close ul after ?


Maybe it would additionally be in order to also mention in the class javadoc that the listener adds its own hard-coded categories depending on the debug / trace properties, just as you did for the spring boot documentation.


Why not an external css file ? More easy to update via template ?


That should be `fill_block()`, not `fill_block_with_xor()` again :)


(it now compiles but crashes somewhere after the animationController::setup()... )


It is added as part of this PR but not in this commit. I'll merge this commit into https://github.com/lutovich/neo4j/commit/33bf4a040887b3fa879b256605b42efbbadf3947


TeamCity Civic Assembly :: Build all [Build 93](http://build.civicconnecttlabs.com/tc/viewLog.html?buildId=1838&buildTypeId=CivicAssembly_BuildAll&guest=1) outcome was **FAILURE**
Summary: Tests failed: 71, passed: 1347, ignored: 3 Build time: 00:32:55
### Failed tests
```
Basic Feedback Message CRUD Actions: Should Delete feedback message: <no details avaliable>

Venue tests: Venue type: Should Not Add Venue if Cancel button pressed: <no details avaliable>

Venue tests: Venue type: Should Add Venue: <no details avaliable>

Venue tests: Venue type: Should Add an Venue and CONTINUE: <no details avaliable>

Venue tests: Venue type: Should Not Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Edit a Venue: <no details avaliable>

Venue tests: Venue type: Should Not Delete a Venue: <no details avaliable>

Venue tests: Venue type: Should Delete an Venue: <no details avaliable>

Venue tests: Conference place type: Should Not Add Conference place if Cancel button pressed: <no details avaliable>

Venue tests: Conference place type: Should Add Conference place: <no details avaliable>

Venue tests: Conference place type: Should Add an Conference place and CONTINUE: <no details avaliable>

##### there are 61 more failed tests, see build details
```


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


Kill this, it's an affectionate idiom.


I don't know how to delete the extra https://www.softaculous.com/apps/others/Mautic written after **disclaimer** text.   Github has no option to 'edit commit' or something similar.


nitpick: can you `public`/`private` these, format them (functions aren't single line in the code base so far) and comment them.

Auto formatting in Android Studio: http://stackoverflow.com/questions/16580171/code-formatting-shortcut-in-android-studio

If you use the shortcut however, be aware that all logging (```if (DEBUG) Log.d("Title", "Message");```) will be formatted as:
```
if (DEBUG) {
   Log.d("Title", "Message");
}
```.

Either format manually or look into editing the auto formatter to ignore logging code. I have not looked into how to do that yet but I will let you know if I figure out how.


This is coming together nicely.  Great work


Not sure if this is overcomplicating things. Whats wrong with the native way?


Kill, this is rare (as determined by a Google search for "veiled ambush").


@SebastianTroc can you describe, step by step, the actions to do to reproduce it? Or provide a small gist with an example?


Kill this. It false alarms on some common usages, such as "Obama's favorable approval ratings".


Although...yes, we really can re-encode it as UTF-8, I missed that part.

Just for clarity:
There is no reason to /not/ do a unicode-lower for non-postgres databases - it's just that for postgres this is required, while other databases are more tolerant. So, removing the case discrimination is correct - and in fact it complies with the unicode guideline and applies it everywhere instead of just for this one case.


KT-9739 Looks important to me


Careful with argument/option confusion for echo and grep, and with quoting in the regex. Hmm, maybe shell script is not the best language for this?


I don't understand this commit. Everything is opposite to conventions already present in lubridate or core R. We are using `tz`, `format` and `origin`.


Wow, I didn't knew that! Thanks for making this core better and more accessible for everyone :smile:


Was that not due to some local configuration issue? Like having not run `npm install`? Not 100% if this should be caught here or allowed to bubble up @dannyroberts


[gogo[](url)]([url](url))


Eek code injection.


i experiencing the same problem, but one thing is more frightening

when making a GET to /service/endpoint1
i received 2x Location: /endpoint1/resource-id1, Location: /endpoint2/resource-id2

the Location headers are returned the when creating a new resource using POST

it would seem that something is thread unsafe


Please add some comment why this test is disabled


@alexkiro I made the changes in 18595a8, unfortunately '__cmp__' magic method is no longer available in python3 but I manged to use the other comparison magic methods ('__eq__', '__gt__' etc)


Code injection.


кто-то хочет получить по жопе?


I explained that in the pull request.


Please change the correct form to "at present the".


vous pourriez mettre à jour votre `.gitignore` (et utiliser `git add -p`, oui, je rabâche)


Apart from security, does this actually work?


I made a pull-request DefinitelyTyped/DefinitelyTyped/pull/8502


I love PHP


Kill this ("Hush, little baby", "At four pounds, seven ounces, John was a little baby").


_removeFile (e) {


I'm not sure if this is a redundancy or a needless intensifier, like "very".


hm and what does that tell you about ` && ` in the expectation?
poke a little around, change the left term, the right term, are you sure it does what you intended?


Commented code


Fixed the re-encoding.

This case discrimination was a bit odd - for postgres we do a unicode-lower, while for mysql we do a str-lower. What's the point? It's the Python str.lower() that doesn't correctly convert İ - and that isn't specific for this one case, but generally everywhere where we want to be case-insensitive.

Sure - if postgres is set up with the wrong locale, then it wouldn't correctly collate either, but there is very little we can do about that in our code (and hence this was not and is being caught). So that belongs into the installation guidelines, right?

For the code, the unicode guideline gives a clear rule: lower() on user strings => always convert to unicode first. And that is absolutely something that must be generalized, independent of the database used, and independent of the specific case.


Kill this. It will false alarm on phrases like "Those who came first began their day with the freshest coffee."


Does it make sense to translate this?


I'd put `m` in its own line


I think this is more of a cliché than a redundancy.


isn't that name a bit generic for a somewhat specific global function?


`callback: applyFilters`?


There's no better way that using a global variable for this? Like passing it as an argument to `applyFilters`?


It's really strange language ;-)


Please remove this "vv" ?


If you're adding code reversed from another game than SQ6, can you please at least make a note of that in the commit messages so we know what it's from? Otherwise it'll be very hard to figure out which code is for which SCI version later, and which parts still need additional reversing for other versions.


У радиокнопок должны быть одинаковые `name`, а отличаться `value`.


change _removeImage to _removeFile


space inside `{{ ... }}`


THANK YOU!!


### `hello`


I would like to get rid of `content_for :javascript`, since that breaks deface overrides - https://github.com/spree/deface/issues/125. Easiest workaround is to put it into a separate partial.


_hello world_


@Patater please add the comment regarding this function beeing replaced in future by  uVisor-provided API


Possible it's a good idea, but unfortunately I will not be able to use this application in *nix environment.


Eh, I don't love this one. There are shades of black and you can't really say "It's black outside" — you'd say it's dark.


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - i386-Windows-7-VisualStudio-12: OK (2129 of 2137 tests passed, 3 test warnings)

 * 10 tests had: EIO diffs.
 * 8 tests had: ESO big diffs.
 * 7 tests had: Table big diffs.
 * 2 tests had: AUD diffs.
 * 6 tests had: ERR diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 4 tests had: MTR small diffs.
 * 2 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


"Stairway to heaven")


The comment said: "Python lower() only works properly on Unicode strings not UTF-8 encoded strings"

...but then does a case discrimination only for postgres. Doesn't make sense to me - it's the same issue for all databases. Just, MySQL and SQLite are more lenient, and postgres, as usual, is strict as hell.

However, the postgres branch is the correct branch - the else-branch is just "being sloppy if not forced to do it right" :D


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


This would fix this issue https://github.com/kwhitley/apicache/issues/18


Тут немного не правильно понял, `.prices__table-header--bold` должен быть элементом, а `.prices__popular-icon` как раз модификатор.


This is often used as a phrasal adjective "burning-hot coffee". I think the replacement should be "hot", not "burning".


This action you should do in finally block or in try() expression.


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18877](https://teamcity.mesosphere.io/viewLog.html?buildId=161025&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:48:18


I think there is a misunderstanding here. If code is 403, `message` will be `TR__ERROR_HTTP_FORBIDDEN`. This is currently translated to "Forbidden", but we can easily change the translation to "You don't have permissions to see this content.". That would avoid adding yet another special case.


Тоже самое с `name`. Модификаторы тут не нужны, все равно они будут скрыты.


I've tested the remapping functionality in all the SCI2 - SCI2.1 games, as well as the GK1 executable. This is the only notable difference for now.
The games I've tested are:
- GK1 (245 - 254)
- PQ4 (245 - 254)
- QFG4 (245 - 254)
- KQ7 (235 - 244)
- Torin (no remapping)
- Shivers (no remapping)
- GK2 (235 - 244)
- SQ6 (235 - 244)
- LSL6 hires (no remapping)
- MUMG Deluxe (no remapping)

That's what I can remember from my checks... is there anything else you need?


годно, одобряю!


A ti si?

Poslano s mojeg iPhonea

10.03.2016., u 18:37, vlajic3 <notifications@github.com> napisao:

> not vorking list
>
> —
> Reply to this email directly or view it on GitHub.
>


Client raises `RuntimeError` when an unsupported version of qbittorrent is used. Will be used as warning to avoid issues like #4.


Ok, sure, I saw that is the convention in the rest of the project.


> This removes tags of the type tm_tag_undef_t. Some parsers like SQL, fortran, VHDL don't have all their tags mapped to geany tag types so these tags become tm_tag_undef_t. Right now these tags aren't saved to the tags files. Just wondering, couldn't they be interesting for us? (e.g. for goto tag definition/declaration, code completion)

Answering myself - keeping tm_tag_undef_t tag types is probably useless because they won't be found by the search functions anyway.


@AMikhniev After `if ... then` nothing should be on the same line. Start from new line.


LGTM


This is complicated. "But however, the man ate the cheese" should become "However, the man …" not "But, the man...", and "but nevertheless" sometimes needs to be rewritten in a more complicated way, e.g., "He ran, but nevertheless was caught" should be "He ran, but was caught nevertheless."

Maybe kill this and open a separate issue.


thanks, Tim


That's not the point at all. Which game is the code you reversed from? How are we supposed to compare interpreters for subtle differences we may run into later, if we don't know the baseline??


I'm not really familiar with C, but why do you first store the formatted string and then print it? Wouldn't it be easier to use something like `printf()` directly?


Great! This marker interface is essential for serialize/deserialize operations.


Ok. This works fine for me as a data- attribute.

On Mar 11, 2016, at 12:51 AM, Rod Roark <notifications@github.com> wrote:

The changes are not needed.

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


Kill this. These are legal terms with different meanings. Cease means stop. Desist means don't start up again.


The code is from SQ6 and Phantasmagoria 1, and some differences from GK1


5113_GHX_8.2-8.3_Transition (mitchute) - i386-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


If your plan is to get the decimal key number out, I think this works. You could use `cut -ds` maybe? It doesn't matter though if there are no surprise lines in `/proc/keys` without spaces. FWIW, I had problems getting the decimal expansion out of the awk command, but I guess it has worked properly. :-) The awk command has also concatenated all key ids it has found, which the grep solution doesn't do.


Please add other forms such as "comprised of".


This function is not required. Please, keep tests as small as possible.
This test requires only declaration of the struct.Node.


I have a complete Emacs Lisp implementation of the Common Lisp reader:
https://github.com/larsbrinkhoff/emacs-cl/blob/master/cl-reader.el


Please also check that the donation batch has been created.


@epsitec this commit should fix the problem on the unix-likes OSes but I can not publish on npmjs (I'm not in the collaborators list: https://www.npmjs.com/package/electrum-require-components)


I think that I may have a solution. The curl command has a flag ( -w) that allows to print some valuable information , like the http-code-response , the total time, the upload time. And in this case that we are using random information to put (from /dev/random), the -w flag with some variables will give us more general information.

i.e.

#FILE=$(head /dev/random -c8 | od -tx1 -w16 | head -n1 | cut -d' ' -f2- | tr -d ' '); curl -s -i -A 'Mozilla/5.0 (X11; Linux i686; rv:6.0) Gecko/20100101 Firefox/15.0' -X 'PUT' -w \\n\\n\\nhttp-response-code:%{http_code}\\nRemote-IP:%{remote_ip}\\nRemote-Port%{remote_port}\\nUploaded-size:%{size_upload}\\nSize-Downloaded:%{size_download}\\nConnect-Time:%{time_connect}s\\nTotal-time:%{time_total}s\\nEffective-url:%{url_effective}\\n  --data "$FILE" -k "https://target.com/admin/$FILE" 2> /dev/null

The part of the output will be:
http-response-code:501
Remote-IP:104.85.166.89
Remote-Port443
Uploaded-size:16
Size-Downloaded:257
Connect-Time:0.074s
Total-time:0.612s
Effective-url:https://target.com/admin/66c00f5789927587


Should it be moved to ``c2corg_api.views.documents`` since it is used both for routes and outings?


Run the script as the administrator~~


http://llvm.org/docs/CodingStandards.html#name-types-functions-variables-and-enumerators-properly
Variable names should be nouns (as they represent state). The name should be camel case, and start with an upper case letter (e.g. Leader or Boats).


Well, it will call terminate on the previous actions until the user stops debouncing... It should only be the last action called for success.


Did you decide that testing the correct creation of donation batches (one per date per venue) should be done separately, and not as part of the integration test?


@radubrehar can't figure out why this error appears:
`Uncaught Invariant Violation: addComponentAsRefTo(...): Only a ReactOwner can have refs. You might be adding a ref to a component that was not created inside a component's `render` method, or you have multiple copies of React loaded (details: https://fb.me/react-refs-must-have-owner).`

I found an answer here https://gist.github.com/jimfb/4faa6cbfb1ef476bd105

But, I don't think there are two react versions included.
The second reason reported is that the component with ref is rendered outside the class (I've changed a bit the code so the object description is passed direct to createClass).


LGTM


ok, then we do that - but the bigger problem is, that we cant show 403 at all as the user gets logged out immediately


This was updated later when introducing a user as an argument to the jar execution 656d0d7d521ca783337a9c5ef4339f50ea901772


styled


я не нашел как по другому оставить коменты, поэтому так, там используется сырой компиляттор а у тебя qt хедеры их там нет, убирай


Updated to reflect recent changes in directory structure.


тоже къют


Hey @cosimoc , where do you suggest I have this function? Because the _getInterestingWindows is in a different class (AppIconButton). Should I make it global in the file or should I have that function in the shell-app itself? (I can do the latter but maybe then it can be confusing to have the WindowTracker considering the speedwagon interesting and the ShellApp not doing it...)


why this space.


Commit explanation:
CNTFRQ is *NOT* writable in non-secure mode so it must be set before entering non-secure hyp mode.


файлик должен называться просто input.txt, лежать вместе с кодом


таже херня


lgtm, maybe add a comment about why the log is turned off and where the -p argument comes from?


это кут, надо просто с++


how about naming this only `create_users`. I think this:

    escalus_fresh:create_users

Is cleaner than:

     escalus_fresh:create_fresh_users

I don't think it's enough to say fresh only in the module name.


hi im currently working on deploying openstack using compass. I have installed compass following the procedure given in the link http://www.syscompass.org/install.html. I was succesful in installing the compass but while trying to deploy openstack i keep getting the error ERROR404  "message :"flavour openstack_kilo:single-controller does not exist","traceback":"none". It would be very helpful to me if u have any solution for this.
                                                Thanks in Advance.


Oh, I think in the end we decide to do `oFontsIncludeAll` when `$o-card-is-silent == false`


Also re-added the comment to explain the problem with postgres on Windows.

As I said - this is how it works, and how it is compliant with the unicode guideline. There is little we can do if postgres has been installed with the wrong locale, and there is nothing we did do about it before I moved the s3_unicode into S3Duplicate. So this change is actually correct, and now applies the solution everywhere not only for one case.


is an actual file needed?


Android an iPhone builds complain about undefined reference to `squish::DecompressImage(unsigned char*, int, int, void const*, int)` here.


If we define MT as:
std::vector<SPIRVType *> MT(T->getStructNumElements(), nullptr);
next two lines will be redundant.


I agree with this. I'm going to remove the deep ref.


Summary of changes:

- Retry on failed poll
- Add wipe/format
- Remove send timeout and print transferred bytes
- Make `send` more verbose
- Add optional second parameter to `send` to set a title
- Extend help


example how i do that? if i just create a `_form.js.erb` or `_form.js.haml` it ignores my file.


because you are doing this at both Read() and ReadAt() and hence doing it here will be redundant


That's because you did not pass the port in the environment. In production you have to pass the environment and port.
`NODE_ENV=production PORT=8081 node server.js`

Let if we it still create an issue.


Yes, that way we differentiate which URLs our app handles and which not.


this is a short stories app


:sparkles:


Could you clarify why openStructType accepts struct member types? They are unknown at this point. We only know how many members our struct has.


цю операцію треба залишити на тому місці де вона була. логічно неправильно підписувати контракт а потім переключати стан тендера. в тесті це працює тільки тому, що ці дії виконуються хаками.


I think we can translate non-recursive types immediately:
MT[I] = transType(pElemTy);

We don't need 'before' vector at all.


This changes the formula in the lib and is not advertised in the commit message.


see [app/views/tasks/](https://github.com/foodcoops/foodsoft/tree/master/app/views/tasks) then `_form` and `_form_javascript`


ця дія має бути нижче. на тому місця де і була.


just by reading the code, it is not clear, that suspend also waits for success. Please rename to `suspendAndWaitForSuccess` according to `create`.


French bashing


thx, but i found it already ;-)


pourquoi ??????


Ok


did not know `withClue`. cool.


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


1. В таблице petitions нет поля для связи с таблицей users.
2. Не авторизованный пользователь может создать петицию, чтобы это исправить нужно в PetitionsController добавить строку  before_action :authorize, only: [:new, :create].
3. Почему закоментирован код
```
# def destroy
  #   session[:user_id] = nil
  #   redirect_to root_url, notice: "Вы вышли"
  # end
```
?
4. current_user.petitions этот код должен быть в контроллере, а во вьюхе только переменная
5. По корневому роуту у вас выводятся все петиции а должно выводить 10 последних


Really appreciate, that you fetch and compare the mesos state!


I mean the following:

A smart DNS template ready for minor configuration and it should be ready to go.  This will spare novice users from going deep with configuration steps where they can make mistakes.

By the way, is your personal smart DNS still working with US Netflix ?

Regards,

Hussain Abukabbos

> On 11 Mar 2016, at 13:05, Anton Belodedenko <notifications@github.com> wrote:
>
> Sorry, I don't understand what you are asking.
>
> > On 11 Mar 2016, at 09:48, abukabhs <notifications@github.com> wrote:
> >
> > Do you have a ready made template that one can copy, upload to a server and slightly modify it with his domain, DNS ip and ISP IP ?
> >
> > Regards,
> >
> >
> > > On 11 Mar 2016, at 01:18, Anton Belodedenko <notifications@github.com> wrote:
> > >
> > > Your guess is as good as anyone else's.. Probably.
> > >
> > > —
> > > Reply to this email directly or view it on GitHub.
> > >
> > —
> > Reply to this email directly or view it on GitHub.
> >
> —
> Reply to this email directly or view it on GitHub.
>


Git Real2


Hello Tobias,

thanks for your effort. Do we need the interface 'ol.format.FormatType'? Currently it is not used and there is no class  'ol.format.FormatType' described in the OL3-API-Docs.

Regards

Tino


These might be cleaner in a try finally. I can understand wanting to do this for consistency but it's actually a no-op because of the writer impl.


Nope. I just checked. Not needed.


[IMO]indexアクションにして、routes.rbでresources使ったほうがRailsっぽいけど、どうっすか？


Thanks!

On Fri, Mar 11, 2016 at 11:33 AM, Frank Denis <notifications@github.com>
wrote:

> That should be fill_block(), not fill_block_with_xor() again :)
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/P-H-C/phc-winner-argon2/commit/c74eecdff1782b67b042c88c476313dfb63830a3#commitcomment-16640715>
> .
>

--
Best regards,
Dmitry Khovratovich


This breaks API compatibility, we might need to bump a minor version.


you should squash these in the previous commit


> the else-branch is just "being sloppy if not forced to do it right"

In general I am all up for making the system do the minimum amount of work required to keep performance optimal


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


@amicojeko @lleirborras this is a rework of 8c3adc1f9854106d90badb999a415ae744c230e4, see fe84c0567f96f1526ed4be341d03a4a83fb37ea7 for the use case that you were solving.

Actually in the Hawk codebase there was already a solution, but it wasn't used on all association types. This change uses the lookup logic on all associations.

The problem with the previous solution was that it is evaluated at _compile time_, where there may be interdependencies between the classes being evaluated, as such trying to `const_get` them forcefully may raise `NameError`.

Consider this example:

```ruby
# Client Library
class Client::Foo < Hawk::Model::Base
  has_many :bars
end

class Client::Bar < Hawk::Model::Base
  belongs_to :foo
end

# App codebase
class App::Foo < Client::Foo
  AWESOMENESS = :high
end

class App::Bar < Client::Bar
  default_scope awesomeness: App::Foo::AWESOMENESS
end
```

With the previous approach, the code tried to find `App::Bar` in the `[inherited](https://github.com/ifad/hawk/commit/fe84c0567f96f1526ed4be341d03a4a83fb37ea7#diff-d3b493d00f3d7a741abd825208796044L73)` callback, while re-evaluating the `App::Foo` associations.

However, when trying a `const_get` on, `App::Bar` as this references a constant within `App::Foo`, that would have raised a `NameError` leaving `App::Foo` associated with `Client::Bar`.

This approach instead does the resolution at runtime, when associations are actually executed and fetched from the remote server. At this point, in production all constants are loaded while in development we can be sure that the *calling* constant is fully defined, and `const_get` will trigger the autoloading chain, and the *called* constant will find the calling one, if referenced.

A cache of the resolution is also introduced for speed.


GCC never complained o.O


@larsbrinkhoff Wow! Nice! The thing I started here is much more limited in scope, though, and it has its place if all you want is to slightly enhance Elisp.


это код в контроллер нужно переносить


Bearing in mind I am using C++11 features if that has anything to do with it


Should be fine with just

`this._url !== undefined`

shouldn't it?


get 'sessions/new'
 +  get 'petitions/index'
 +  get 'petitions/new'
 +  get 'petitions/edit'
 +  get 'petitions/destroy'
 +  get 'petitions/show'
 +  get 'users/index'
 +  get 'users/new'
 +  get 'users/edit'
 +  get 'users/destroy'
 +  get 'users/show'

эти все роуты у вас же заменяет две строки
+resources :users
 +resources :petitions


TeamCity jonas-requests :: WindowsBuild [Build 35](https://build.neohq.net/viewLog.html?buildId=575379&buildTypeId=JonasHaRequests_WindowsBuild) is now running


That was a mistake, have changed it to return NULL.


Brady what about if we add method "smarty_make_timestamp" inside calendar with changed name like "calendar_make_timestamp"?


@nk0leg, maybe it worth to exit method in case of absence of the context?


This line is wrong:
var program_files = is64Bit ? 'ProgramFiles(x86)' : 'ProgramFiles';
ProgramFiles is used for x64 programs.


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


этот код должен быть в app/views/layouts/application.html.erb


makes sense, do tests still pass?


http://stackoverflow.com/a/15013295
Yup, GCC allows it. I will have to correct this for other compilers :)


в контроллер User.find_by_id(petition.user_id).email


Or other solution is to change date format for month and year e.g. (%Y/%m/%d) with 'forward slash'


agreed!

On Fri, Mar 11, 2016 at 12:10 PM, Michał Piotrowski <
notifications@github.com> wrote:

> how about naming this only create_users. I think this:
>
> escalus_fresh:create_users
>
> Is cleaner than:
>
>  escalus_fresh:create_fresh_users
>
> I don't think it's enough to say fresh only in the module name.
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/esl/escalus/commit/ed23eee96fa204d806231cd1dcead9f22dc14e3e#commitcomment-16641383>
> .
>

--
Simon Zelazny
Erlang Solutions, Ltd.


отступ должен быть везде 2 пробела


TeamCity OSS :: Kubernetes Mesos :: 4 - Smoke Tests [Build 18883](https://teamcity.mesosphere.io/viewLog.html?buildId=161045&buildTypeId=Oss_KubernetesMesos_5SmokeTestsDockerMesos&guest=1) outcome was **FAILURE**
Summary: Exit code 1 (new) Build time: 00:13:17


At some point I thought it would be a good idea to separate request errors (connection issues) with request failures(failure http status codes).

So we the requestError() and requestFailure() methods which log a slightly different message and also the later is trying to display the server status.

Moving everything inside the first try/catch block we hide everything behind request error.


Just curios, is the infinite loop message causing a problem with the plugin or did you remove it for other reasons. If there's a problem with it let me know what I'll see what I can do to fix it.

~JH


IMO, this test should use `self.cache.set('prefix_key', 'foo')`, otherwise it does not test much.


Thanks For great information.
Regars
Rajmi
Marketing Head
[https://www.canadapharmacyexperts.com/](https://www.canadapharmacyexperts.com/)


What if it's a _relative_ path to another, non local rule set, like:
../../../../../Program Files (x86)/../MicrosoftManagedMinimal.ruleset


Sorry, but you have to follow the requirements in Project Sunshine.

On Fri, Mar 11, 2016 at 4:24 PM, KC <notifications@github.com> wrote:

> Sir good afternoon. Regarding of project Sunshine sir, I would like to ask
> if is it okay to add some features to the application, but on some cases,
> the changes can affect the project sir, especially the User Interface. The
> goal of the project Sunshine sir is in there, but I want to add the list of
> location and other things. Is it okay sir? Or do we need to follow what is
> on the tutorial. Thank you sir. God bless
>
>
> On Sun, Jan 10, 2016 at 2:34 AM, Ahdzlee Formentera <
> notifications@github.com> wrote:
>
> > Please also improve the commit statements. Be at least be more
> > descriptive. At least state briefly what you have done on this stage of
> > commit.
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <
> https://github.com/KCArante/SolarSystem/commit/20015b3d1a7a981795b2f155e6f4192ce35c0bd7#commitcomment-15356188
> >
> > .
> >
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/KCArante/SolarSystem/commit/20015b3d1a7a981795b2f155e6f4192ce35c0bd7#commitcomment-16638204>
> .
>


An issue is here!


Oh, this'll end up below!!


Oh, this'll too end up below!!


What is the purpose of this ?


+1. though i'm not sure why the form and JavaScript is not parse from template file :)


borrowing  patch from here : https://github.com/buildroot/buildroot/blob/master/package/bluez5_utils/0001-tools-avinfo-Fix-big-endian-build.patch


What is the purpose of this test?


ditto: `self.cache.set('prefix_key', 'foo', 10.0)`


"Sorry" ? :smile:


This comment isn't relative; case changes during loop.


Yeah - but in this case it's wrong ;) str.lower() does /not/ correctly convert certain unicode characters - and that does not depend on the DBMS used. Leniency of the DBMS is not guaranteed, and in fact, violates unicode rules, so this leads to unpredictable behavior.

And then - we're trading one thing for another here: saving the overheads for the settings lookup vs. saving the overheads for the unicode conversion - I'm not sure that this really changes anything.


The form was rendering, the issue was you had no way to perform options on the form once it was rendered. So we had a few things that had to be setup in the form but it couldn't be done as we didn't know when the form was on the page. Now you have a rendered callback :)


Enabled flatpages (https://docs.djangoproject.com/en/1.9/ref/contrib/flatpages/)

@davidhadrianus89 @novikesumaningtyas please run:

* python manage.py migrate


Great to have a Mac user working on this, too... should we throw stuff together and maintain one repo instead of multiple forks? Cheers from Germany.


naina agarwal


Remember that both actions are string operations ;)

With the additional case discrimination, we have double overheads for postgres and single for other DBMS - without it, we have a correct solution and single overheads for all DBMS.


@mythi Do you mean I need to combine the two commits into one?


toll


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


i have used `key` instead of `prefix_key` because prefix is being passed while calling the function and its being concatenated with `key` in `def get()` and `def set()`


colorChanged is an array of booleans, so this is wrong. (Also in many other places.)


нужно внести папку .idea в файл .gitignore чтоб не было мусора от ide в репозитории с проектом


this is not a test, its a generic function used to get backend with some prefix key in all tests.


For using `--stdin`, I have added `-use_stdin = True`. Do I need to use in `arguments` as well ? Also this is how I am running the bear for `stdin` but it seems wrong to me.

`coala --bears=RuboCopBear < bad.rb `


I realise that but my guess (not tested) is that a unicode conversion is slower than an an if settings.get()
Anyway, not too huge a deal either way I'm sure & it's true that Postgres is our default production DB for which optimising is a good idea


You should not edit anything in jitsi-meet in this case:
1) register on https://en.gravatar.com/
2) upload your avatar there
3) set your email address in the jitsi settings
Jitsi will use your avatar from the gravatar service ;)


Variables to set:
BR2_PACKAGE_CPPSDK_MQC
BR2_PACKAGE_PLUGIN_HORIZONCOMMUNICATOR
BR2_PACKAGE_WEBBRIDGE_TESTMQC


Just curious, where does `byCreatedAt` come from?


**SUCCESS** - TeamCity HotRod C# Client :: Release Windows 2 [Build 15](http://ci.infinispan.org/viewLog.html?buildId=37335&buildTypeId=HotRodCSharp_ReleaseWindows2&guest=1) for 6ee9b91732e2f108349ff583e30b5c3bbfbad717
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:12:31


ah, found it :)


@AMikhniev After if ... then nothing should be on the same line. Start from new line.


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


test


Huuuh? Isn't it already in the code?

    "devDependencies": {
      "babel-preset-es2015": "^6.6.0",
      "babel-register": "^6.6.5",
      "gulp": "^3.9.1",
      "nearley": "^2.4.0" <-----
    }

Going to test with a fresh clone on my end.


ja sam koristio ovu listu dva mjeseca radi samo na njoj dva kanala samo sportska tv
ja sam dragan vlajic


@matt-keibler I thought I'd get your opinion on this: I was doing more RDD (Repl driven development if that's a thing) here instead of working this out in my unit tests. I'd like to work this out more in my unit tests instead. My hangup here is, I call into Slack.channel_history/1 and Slack.slacker/1 - a module which calls into the Slack API.

I thought about "injecting" these functions by making them parameters to the function but was iffy about that. I haven't played around much with any stubbing so I thought this would be a good time to learn about that and wanted to see what you thought.


@mhardin we have split the method and wrote 2 unit test to cover that!  'null' is taken care as well!


Nice addition. Do you plan to push this change back to the arff project?


Test


Sorry, spied on you... ;) Interesting... you co-published with Dourish...


do you mean to use `self.cache.set('prefix_key', 'foo')` instead of `backend_prefix.set('key', 'foo')` ?


нужно начать с этой строки


Works for me..

```
$ git clone https://github.com/liam4/nearley-test.git
$ cd nearley-test
$ npm install
$ gulp
(success)
```


不要慌啊好好学，这里你们就放心吧XD


понял


Weird indent here :)


+1 Will copy ;)


Date formatting not working properly i'm trying some other solution


удаление произошло поэтому строка красная


I found this article about it: http://www.devexp.eu/2014/10/26/use-unsupported-jenkins-plugins-with-jenkins-dsl/
but in fact I used shell step to manipulate terraform, this was easier and faster to our unstable goals.
Thank you!


@nk0leg, you've messed **NOT** in the if condition. Also, please fix the log message. remove extra space in "context  is", add space before "exit".

Are you sure that this is error?


@matt-keibler I wonder if we should bring up the ``` ^ ```  operator too?
```elixir
iex(3)> foo = 1
1
iex(4)> ^foo = 2
** (MatchError) no match of right hand side value: 2

iex(4)> foo = 2
2
iex(5)>
```


See https://github.com/fabric8io/kubernetes-client/pull/330/files#diff-bd0b3215ca04a57dd5925ca30802efd4R219


@thet Jenkins CI reporting about code analysis
See the full report here: http://jenkins.plone.org/job/package-plonetheme.barceloneta/19/violations
```
plonetheme/__init__.py:0:1: C101 Coding magic comment not found
plonetheme/barceloneta/__init__.py:0:1: C101 Coding magic comment not found
plonetheme/barceloneta/interfaces.py:0:1: C101 Coding magic comment not found
plonetheme/barceloneta/interfaces.py:0:1: I001 isort found changes, run it on the file
plonetheme/barceloneta/interfaces.py:7:8: W292 no newline at end of file
```
Follow [these instructions](https://github.com/plone/jenkins.plone.org/blob/master/docs/source/run-qa-on-package.rst) to reproduce it locally.


Thanks @sohje as always


Broke https://github.com/LimeSurvey/LimeSurvey/commit/1dd87f28fafea264630d7093143d8447d2cd8610 again


Thanks @sohje


why there is need to sleep for 0.02 sec when we are expiring it in 0.0 sec?


@forresst :+1: :yum:


Won't this run `celery` _after_ the tests are run?


You could use
```py
get_queue(self, queue_name, **kwargs):
```
And then check for `"serializer" in kwargs`, but that might be overkill.


:shipit:


ok :)


这有啥6的


5-ый коммит


87382760-VAVTerminalUnitSchedule-Issue4285 (mjwitte) - Win64-Windows-7-VisualStudio-12: OK (2129 of 2137 tests passed, 3 test warnings)

 * 10 tests had: EIO diffs.
 * 8 tests had: ESO big diffs.
 * 7 tests had: Table big diffs.
 * 2 tests had: AUD diffs.
 * 6 tests had: ERR diffs.
 * 2 tests had: MDD diffs.
 * 2 tests had: MTD diffs.
 * 2 tests had: RDD diffs.
 * 3 tests had: ESO small diffs.
 * 3 tests had: MTR small diffs.
 * 1 test had: Table small diffs.
 * 4 tests had: MTR big diffs.

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


I don't know enough about the problem to recommend that, config settings shouldn't be changed outside of bootstrap


I do not feel different about the index either way. At some point it will have to be read and presented to the user, and in that case will have to be 1..numPages. Perhaps using another variable name (eg pageIndex) will clear up this type of confusion?


See commit https://github.com/lausser/GLPlugin/pull/1 where I added the managementServer OID in the other repository (GLPlugin). Actually the name is mgProdName, I will adapt it.


@AByzhynar changed in https://github.com/AMikhniev/sdl_atf/commit/dfea13a26b8810e489b7027ea3ec393ce07ca6c5


How's the "Player" going? :)


Depends on whether value is a str or a unicode object.

- If it is already unicode, then s3_unicode is 4x as fast as the settings lookup.
- If it is a str, then s3_unicode is 25% slower than the settings lookup.

However,
- we're talking about sub-nanosecond differences here, which, compared to the execution of the query itself is absolutely nothing. It will be half a second when you deduplicate 10 million records, but at that scale the import itself will take hours ;)
- for postgres, we almost double the effort, i.e. the case discrimination /plus/ s3_unicode, making 175%. That now may become measurable at 5 million records already :D

So I think, it's correctness here that counts rather than performance - although in general I agree that we shouldn't do unnecessary things if we can safely avoid it.


Sorry, i should create a new pull request, i tried but failed again, seems i need create new branch in github first.

This modification is optional, It's concerned with https://github.com/sjitech/rethinkdb/commit/1d442c39557889b4cf639112dfd2d0598c0b4e28#comments.

Suppose you have an object with many properties, i do not want null properties be saved to db, so using a annotation @NotSaveNull seems be a better way. When not specify @NotSaveNull, it acts normally: save null to db.
```
import com.rethinkdb.annotations.NotSaveNull;

    @NotSaveNull
    public class TestPojoIgnoreNull {
        private Object notNullProperty = "foo";
        private Object nullProperty = null;
        ... ...
    }
```

The tested case have been created:

testSaveObjectIgnoreNullProperties
testSaveObjectIgnoreNullPropertiesNested


You have a typo here...it should be THREAD_FACTORY that is assigned a value, not threadFactory.


I see, ok, I'll try to make adjustments to your version to be usable for regular Docker users if I'll get time some day.


Nice


Nein, wurde er leider nicht.


Its a typescript convention to use arrow functions (lambdas) over anonymous functions. Lambdas and anonymous functions in particular also shouldn't be longer than 10 lines (even more so than named functions) as what they do stops being obvious at some point. I know it might feel pointless to enforce this rule here but do give it a shot.


Good point, I will try to remove that.


oops, looks like I forgot to push them :/ will update the pull next week.


Won't this trigger too when `new_message_length == 4`?
Probably wanted an `elif` here I think.


If I remove it, we have failures in test:

```
======================================================================
FAIL: test_prefix_value_used_with_expire (cliquet.tests.test_cache.MemoryCacheTest)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/rhubscher/mozilla/cliquet/cliquet/tests/test_cache.py", line 210, in test_prefix_value_used_with_expire
    self.assertIsNone(retrieved)
AssertionError: 'toto' is not None
```


Code seems to be working (when the fix I mention in my other comment is implemented).


no. just the '0655 -> 0644' change belongs to the previous commit


Grrrrr that'll teach me to copy-paste instructions from [another project](https://github.com/delba/Sorry)!
It has been corrected in 95b625dd997f65cbe967ad474325c822f514dbbf.
Thanks :smile:

/cc @onmyway133


This Github drives me crazy. I don1t know if I uploaded my file correctly according to the instructions of the prog assignment. but the program works in my desktop.


Oh clever way to do so :+1:


Are we guaranteed to have `new_message_length == 2` here?
What about
```python
elif new_message_length == 2:
    queue_name, data = new_message
else:
    raise ValueError('Expected such and such message contents, didn't get it')
```


Hey  Tino,
i removed 'ol.format.FormatType'...I'm so sorry
Best regards - Tobias

2016-03-11 12:23 GMT+01:00 TDesjardins <notifications@github.com>:

> Hello Tobias,
>
> thanks for your effort. Do we need the interface 'ol.format.FormatType'?
> Currently it is not used and there is no class 'ol.format.FormatType'
> described in the OL3-API-Docs.
>
> Regards
>
> Tino
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/AAauAAau/GWT-OL3-Playground/commit/c4d3f5003fda90854b2d8965f421c426ab003313#commitcomment-16641604>
> .
>


Yes looks good


Thanks @BridgeAR ! (cc @rbaumier)


looking good!


你好，麻烦再能增加下，如果自己关闭聊天，或者对方关闭聊天了，js能返回个值吗

同时 js 怎么可以关闭聊天


according to pep8 class declaration should occur after leaving to blank line. you need to add one more blank line.


I did some testing and this breaks `zfs userspace` until UPGRADE ioctl is called.


ikke java util


Line is longer than allowed. (81 > 80)

*LineLengthBear, severity NORMAL, section `linelength`.*


Does this fix ::QueuePacket crash?


Where would be the best place to report a bug to switch away from md5 for `download.lst`


magic numbers :(


Hey AlexT, I now have an account. I don't know what I'm doing, but here I am.


This should be using the mraa API not termios


Yeah sure just pass it into create like `create(:checked_url, created_at: 1.year.ago)`


I've tried to use this script but I keep getting a very dumb error and I don't know why... so frustrating

I get "Missing ] after element list. (line 55, file "Code")"

And this is what I'm requesting in API_Parameters

'Signups' : [ 'Signed up', '(properties["country"])=="denmark", 'general', 'day', 'properties["last_touch_utm_campaign"]'],

Any idea why this is happening?


Oh wait I see what you mean. Let me try


5113_GHX_8.2-8.3_Transition (mitchute) - Win64-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


Try to write more meaning full comments. Think of it as explaining it to some one else but should be brief.
'''
A document summarizer class that represents a
document in bag of words form.
'''


Every save call should end in a new line, so adding the new line at this points does create an useless newline.

Now the output looks like this, which is not as kicad does:
```
  ...
  (pad 2 thru_hole circle (at 0 2) (size 1.35 1.35) (drill 0.8) (layers *.Cu *.Mask))
(model Pin_Headers.3dshapes/Pin_Header_Straight_1x02_Pitch2.00mm.wrl
  (at (xyz 0 -0.03937007874015748 0))
  (scale (xyz 1 1 1))
  (rotate (xyz 0 0 90)))

)
```
You have to add ```\n  ``` before the last bracket of _saveModel, and add two more spaces at the beginning of  every line in _saveModel to have the same output as kicad does.


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


Why not:
```php
$this->exec('INSERT'.($ignore !== false) ? ' IGNORE' : ''.' INTO "' . $this->prefix . $table . '" (' . implode(', ', $columns) . ') VALUES (' . implode($values, ', ') . ')');
```


Hmm, `YAML::XS` isn't used once in my module's code. Shouldn't this be a dependency of `JSON::Validator`? I might have missed a `use Swagger2::SchemaValidator` :-/ According to errors like [this on CPAN testers](http://www.cpantesters.org/cpan/report/68f5edfe-e74f-11e5-9391-19ac8fb2e322) `YAML::Syck` would also fulfill the requirement.

What do you think?


How about using a `CheckedUrl.new(ok: true)`? Just nice to use the actual thing if it's possible.


it's w3c valid for now.
Using external CSS and JS for question type rendering will be a second cycle of dev.


count.py:3:1: E302 expected 2 blank lines, found 1
count.py:6:6: E111 indentation is not a multiple of four
count.py:8:6: E111 indentation is not a multiple of four
count.py:8:15: E225 missing whitespace around operator
count.py:8:18: E261 at least two spaces before inline comment
count.py:8:19: E262 inline comment should start with '# '
count.py:9:6: E111 indentation is not a multiple of four
count.py:12:12: E111 indentation is not a multiple of four
count.py:14:12: E111 indentation is not a multiple of four
count.py:16:6: E114 indentation is not a multiple of four (comment)
count.py:17:6: E111 indentation is not a multiple of four
count.py:17:15: E231 missing whitespace after ','
count.py:24:1: W391 blank line at end of file

fix these errors.
whenever you indent use 4 spaces(for indenting).


word = word.lower()
# case folding.


latex-documentation (Myoldmopar) - i386-Windows-7-VisualStudio-12: OK (2135 of 2135 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)


**SUCCESS** - TeamCity HotRod C# Client :: Release Windows 2 [Build 17](http://ci.infinispan.org/viewLog.html?buildId=37346&buildTypeId=HotRodCSharp_ReleaseWindows2&guest=1) for 6ee9b91732e2f108349ff583e30b5c3bbfbad717
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:11:36


(I haven't been running `npm install` because I thought we used bower to manage node packages (etc.) now -- didn't realise we use both. I'll add it to my "update" script. So that's sure to be the origin of my error.)

That said, we seem to check only for exit codes 0 and 1. Are those the only codes xpathValidator returns legitimately? I tried to find out more but I couldn't find its project -- just loads of projects that aren't it. When we validate the app from the deploy page, the 500 isn't presented to the user, but when you open a form, they'll get a Sad Danny if xpathValidator returns any other exit code. So I figured logging the exception would be safer. We could do more, like a soft assert maybe.


ok fine


Caterina hai sbagliato a scrivere


typo here?


I think any internal representation and calculation should always be in the 0 to max-1 format for the sake of other developers. At the point where items are displayed you can use pageIndex+1. Neither way is technically wrong but I'm fairly sure most developers would assume this is how it works.


Interesting idea!

(And I wonder if ultimately we could unify orders and deliveries functionalities.)


Yes @bcamper. this seams fine... but there are other styles where I was mixing filters... that now don't work because the mix happens before the color is set. The only way to solve this issues is adding a imperceptible light... That have some tiny performance lost


Sorry I forgot to update it on the github repo
I'll do it after I come home from school

Sent from my iPad

> On Mar 10, 2016, at 9:29 PM, Ricky C. <notifications@github.com> wrote:
>
> A few more things for homework #1.
>
>  Looks like I'm getting a Uncaught SyntaxError: Unexpected token var. Check the console.
>  It should only allow the top ten highscores
>  The top three scores should be distinct from one another
>  There should be a button that fills in the score randomly.
> —
> Reply to this email directly or view it on GitHub.
>


here is another example https://github.com/tangrams/tangram-sandbox/blob/gh-pages/styles/grain-roads-area.yaml#L284-L295


if word not in wordcount:
    wordcount[word] = 1
        else:
             wordcount[word] += 1

instead of writing this you could also use:
wordcount[word] = wordcount.get(word, 0)

wordcount.get(word) is equivalent of calling wordcount[word] but when you pass second argument it takes that as default value if the key doesn't exists in dictionary.


please fix alignment


test igen


i don't think so, since they have different purpose. for a delivery you do not have a order. you just get the articles (at least for my understanding of the feature)


Didn't work locally and I try to see if the fresh Travis install runs it correctly


Overall, good performance for a beginner. Keep if up :)


vi prøveg inge


Merged in 9c4d9d5c2082dfbd807159c3f60ec0471bf6ec51


Explicit ```pip install numpy``` should remain here. In general building scipy will fail if numpy is not installed (and a wheel file is not available from pypi or local cache).


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


Yes, removed https://github.com/ArmaLife/Altis/pull/363. This [one's still](https://github.com/ArmaLife/Altis/commit/fa62b076d720fcabdd483e18620961f9bb2bc2e6) a work-in-progress.


统一


Here you probably wanted `post_state ERROR`. This made me wonder: you use two clients, one for GET and one for POST only to get a better error message? Isn't this wasting resources?


its the name given to the property that gives the canvas to the leapmotion module, this can me sent null .


then what is the use of `backend_prefix` ?


It would fail here with KeyError


I'm thinking of at some point adding a state after closing the order, where the order person can still update the amounts (including adding articles for stock or extra margin). If there then comes an option to create a new order and close it right away, and the option to create stock articles from an order's left-over articles, then this is basically a delivery.


TeamCity LabKey :: 16.1 Release :: Community :: CDS postgres 9.5 [Build 46](https://teamcity.labkey.org/viewLog.html?buildId=309823&buildTypeId=LabKey_161Release_Community_CdsPostgres94) outcome was **FAILURE**
Summary: Tests failed: 10 (2 new), passed: 42 Build time: 01:31:17
### Failed tests
```
org.labkey.test.Runner: org.labkey.test.tests.cds.CDSVisualizationTest: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyPlotToolTips: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyTimeAxisWithMultipleSchedules: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifySubjectCountsWithFiltersAdvancedOptions: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyGutterPlotBrushing: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyTimeAxisFilter: <no details avaliable>

org.labkey.test.tests.cds.CDSVisualizationTest.verifyLogAndLinearScales: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.testLearnAboutStudyProducts: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.verifyFilters: <no details avaliable>

org.labkey.test.tests.cds.CDSTest.clickOnLearnAboutStudyProductsItem: <no details avaliable>

```


That's what we want to test actually, that the key is correctly updated.


Hi Simon,
MATLAB has the built-in filesep function that aims at returning a platform independent file separator.
I am afraid that your forward slash will fail on windows machines. Better still (for readability) would be to use fullfile()
Can you revise the relevant piece of code and resubmit your pull request?


Good point. I am not sure yet whether I would want to put it here or in the matching section with a reference to it here. I also want to go into more detail about the difference between matching and classical variable assignment with respect to immutability.


e.g. if exist(fullfile(pathstr,[name,'.jpg']))


There are some cool glitches... this is LineOfSitght : )
<img width="1228" alt="screen shot 2016-03-11 at 7 20 30 am" src="https://cloud.githubusercontent.com/assets/346914/13702079/1a119dc2-e75a-11e5-90fc-7f538a7e94e5.png">


**SUCCESS** - TeamCity HotRod C# Client :: Release Windows 2 [Build 18](http://ci.infinispan.org/viewLog.html?buildId=37349&buildTypeId=HotRodCSharp_ReleaseWindows2&guest=1) for 6ee9b91732e2f108349ff583e30b5c3bbfbad717
Tests: 0, 0 failed (0 new), 0 ignored. Build time: 00:11:29


@lauraluiz it fails for me with https://gist.github.com/schleichardt/ce9e9845e136472d68d8


ack


@leplatrem `**why do you have to test if prefix is present?**


maybe it is the api


i dont know why this is not taken out of the core.. since the complete questchain is already scripted in SmartAI???
https://github.com/OregonCore/OregonCore/blob/b5668d7465f6457cbee6f9cab04eccd17821a761/sql/updates/world/2016_02_06_ring_of_blood.sql


ここで、 `request.url` とすると、

```
=> "http://test.host/events?content=%E3%82%A4%E3%83%99%E3%83%B3%E3%83%88%E6%9C%AC%E6%96%871&end_time=2016-04-02+00%3A24%3A35+%2B0900&name=%E3%82%A4%E3%83%99%E3%83%B3%E3%83%88%E5%90%8D1&place=%E3%82%A4%E3%83%99%E3%83%B3%E3%83%88%E9%96%8B%E5%82%AC%E5%A0%B4%E6%89%801&qualify=%E5%8F%82%E5%8A%A0%E8%B3%87%E6%A0%BC&start_time=2016-03-31+21%3A24%3A35+%2B0900"
```

`create` アクションに対してリクエストを送っていなくて、原因がわからなくて詰まっています。
一旦 `post :create` とはせずにURLを直打ちとかしてみたいと思います。


Damn autoformat. But I guess these are not so bad?


not related


We need to check for float/int/double conversion, here the three are mixed, even if it doesn't cause numerical error, it can have some impact on performance. (It's also my fault)


"Главная
Отображает последние 10 петиций."

эм, я ссылался на это требование при формировании запроса


Where are all those ``icomoon`` files defined? It seems they are not included in the PR.


lo possiamo far diventare parte di `Radical.Windows`?


It is the API, maybe the feature is not yet released.


请问大神，，后端的ForumApi.zip文件在何方？


please no un-conditional rescue.

Also, I wonder if this is not a bug which should be fixed elsewhere. Was the DNS server not reachable or what error did you see?


You missed 2a3d2a9074d26f01d4230de974c7717acdd56c2f, which fixes the rendering


Yeah, that seems to be a beneficial improvement. We're talking that over on the forum here: https://forum.pfsense.org/index.php?topic=108116.0


since this is just fixing the former commit, please merge them together before creating a pull request.

You can do so via

 git commit --amend

or now afterwards with

 git rebase -i


I approve last message


@MartijnKaijser thanx! I spend lot of time getting a glue about how all this w32 installation works.


The problem is when the application stays in the backgound for a while and a recently displayed activity is recycled by Android. Bringing back the application causes NPE as the activity's component is called by the fragment before it is initialized. This scenario is equivalent to one when you do not set `setRetainInstance(true);` in your fragment (e.g. you are extending `YouTubePlayerFragment`). It might be related with using `FragmentPagerAdapter` but it's hard to reproduce so i can't really tell right now.


controller input pr is in, in all likelihood this was included, but its worth checking to make sure i did it correctly


可以直接使用tool  debounce方法


"Доступно только залогиненным пользователям. Ссылка на создание петиции должна отображаться всегда, но если пользователь не залогинен то она должа вести на страницу"
Это все требования касательно создания петиции для незалогиненного юзера. У меня этот функционал реализован без метода authorize


hm, it crys for a security issue. Okay, only in the test suite, but nevertheless, please use Dir.mktmpdir instead.


**Code Quality - Nitpick**
You can create all these getters/setters using one `attr_reader` like this:
 ```
class Result
  attr_reader :name, :guess, :time, :chances
end
```


c'est l'ID des tickets orateurs si je ne me trompe, j'ai volontairement ajouté les orateurs et la thym dans les participants pour donner une vue d'ensemble des participants. L'ID actuel c'est celui de la soirée, mais on est pas obligé de filtrer du tout en fait.


This is not nice :(


Ah, thanks!


Where does this come from?


don't you use `=>`?

I'm used to this:
```jsx
const MyComponent = (props) => (
  <div>
     <h1>Hey</h1>
     { ...props }
  </div>
);
```


Might be a good idea to have rgb_from_linear and rgb_to_linear, with a default implementation, within the TransferFn.


This should go to the log file instead.


Thanks for this!


a repo with my favorite lyrics


This looks weird.


endline incorrect -- test --


Delete unused parts.


Maybe we should restore it?


This should be in configuration file.


Here you change the shell to the system version. For the homebrew version, it should be:
```
chsh -s /usr/local/Cellar/zsh/<version>/bin/zsh
```


What's the difference?


shouldnt this be in it and not left out?? its now not registering isnt it?


i'm fine with just 200 for now, the docs weren't clear about this but i've only seen 200s and i'd like to be able to get an email for anything else so i can act on it


Merged in b8010fedca4e3295e303ac99d16291446cd7498f

Note: I am considering adding a `show_group` or so (like `show_user`). In the foodcoop-adam fork, I use this to add a dagger symbol after deleted groups, for example.


`</dt>` missing here and below


Here is calendar date issue fixed without modifying smarty code please review and let me know


What about missing/unreadable configuration file?


もしかして `before_action :authenticate` をやっているから、その部分の定義が抜けているのかも？


So first idea of the top of my head if use the code tag. Because that would look neater. If you wish to use bullets, the correct structure would be:
```
        -  node

           -  child
           -  another child

              -  child of child
              -  last child of child
```
and not what you've shown


For the record, run takes host, port, and option for enabling the integrated debugger. Take a look at the docs!


i like the idea too


`!#` missing


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


Thanks. Updated also explicit `pip install scipy` for scikit-learn.


@leplatrem because in this case `prefix` is not required.


Is it allways second item? Document magic constants.


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


Is it possible to use strings instead of integers for the command line?


Typo?
It should be `heading-6` here.


On ne push pas sur master !


Remove unused file


Got it, done


Don't use word "only". Just document what is does.


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


I've just checked all them, and they are still the same (as in 12.1):
https://github.com/CyanogenMod/android_device_samsung_qcom-common/blob/cm-13.0/cm.dependencies
So it should automatically be added as you said.


Double word


just concise and ES6 syntax. You use arrow functions everywhere else.


yes i saw and dropped it from my PR as that at least force includes it in the installer which preferable i think


Better: multivalue CA names.


Лучше элемент index-title__bem, а не отдельный блок strong.


alignment


maybe rename as `file_md5` or `md5digest`


Зачем модификатор у page?


reuse Utilities method


in case you need String representation override ToString()


Thanks for testing the patch, Pavel. Can you please describe the issue in detail? I'd expect that if this feature should be totally transparent to userspace until it's enabled explicitly.


typo.

Consider mentioning deliver.

E.g. "keep track of md5s allows to skip previously uploaded in deliver"


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


> Они просто гении маскировки... left_ads сменили на ads_left

_(c) EgoAkk_


It doesn't really make *that much* of a difference here, but if you want to go ahead and rewrite them to the arrow function syntax!


are we sure the screenshot doesn't have another way of being type identified ?


License system for Packs


Makes sense. I moved it to `models.utils`.


Здесь надо использовать функцию freeze:

```js
    url: freeze('../../static-ru/nav_methodology.ru.svg')
```

Имя файла: promo-crossroad_methodology.svg


no "main" method has to be defined here


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


It doesn't bother me lol, was just asking.


```js
url: 'methodology/'
```


right, it should be forced


```js
url: 'platform/'
```


```js
url: 'community/'
```


aqui esta legal! bacana.


faltou identar :(*


I don't like commented code like this... and few other instances in this PR.


Этого не надо делать, это делает функция freeze (см. коммент выше в index.ru.bemjson.js)


@AbdealiJK  What is code tag ?


Careful! This is probably going to break for single language sites. I also think we should somehow implement this in the form::serialize method instead of the controller, so it will be reusable.


Oops? Don't think that's the default for everyone


@stereodenis Can you provide more details on the old issue or open a new one? Would prefer a new one as https://github.com/facebook/react-native/issues/5958 is already closed


@e0d and @arbabnazar please review. NOTE: This commit is NOT ready, Just pushed it for early feedback. Still need to make some changes and validate properly with AppNeta access_key


@duftler


no "System.out.println" call should be in the program


Lol party!


An issue is here!


Here in line 4, there's a spelling mistake!


Oh, this'll end up below!!


Oh, this'll too end up below!!


@giuseppec
Are you sure? You need to add to SUGGESTS then.


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


Truncate partitions also causes an error so something like

                SQL_BLACKLIST.add(Pattern.compile("^ALTER\\sTABLE.*TRUNCATE"));

Fixes this.

Richard


Maybe SGE PE?


We don't need this utilitiy function.

`screenshot. device_type`

See https://github.com/fastlane/fastlane/blob/a6641e6e4b990cbf7e421384ada0f11b21d42930/deliver/lib/deliver/app_screenshot.rb#L51-L64


"student1" variable name should be avoided


Maybe we can try firstly the autoconfig of the client
```
client, err := client.InClusterConfig()
if err != nil {
	config := &client.Config{
 		Host: provider.Endpoint,
 	}
}
```


Un día que me dio por ahí jaja


Yes, that works. I pushed an update.


I took the `panel()->site()->language() != panel()->site()->defaultLanguage()` from your implementation in `panel/app/forms/pages/edit.php` (line 44). I think it will prevent single language sites, but if you think it's dangerous, we should better think of something else for `panel/app/forms/pages/edit.php` as well.


how come, this can be omitted now?


this class should be "pojo" class. Just a story for a student representation.
https://en.wikipedia.org/wiki/Plain_Old_Java_Object

the same the read https://en.wikipedia.org/wiki/Single_responsibility_principle.

In this implementation Student class
1. store the "student properties"
2. create the "student" object
3. show information on standard output
4. run the program


What Vinicius wrote is more aligned with what I was trying to say. You could move things around a bit, like use do-while and change the condition to be only the check for memory, then move the last block outside.

The key point is that you wouldn't need macros, because the type specific code would be reduced to the two liners for each case in his example.


I used int, since the other parameter are also using int.
If we want using string, it would be nice to also change the other -r, -t in order to have a consistent set of options.

Something like that could work (but we must implement one per option type)
 ̀``
#define STRENUM(x) if (value == #x) { *type = x; return true;}

static void UpperCase(string* input) {
  std::transform(input->begin(), input->end(), input->begin(), ::toupper);
}

bool StringToXType(string value, XType* type) {
  UpperCase(&value);
  STRENUM(NONE);
  STRENUM(ADJUST_FOCAL_LENGTH);
  STRENUM(ADJUST_PRINCIPAL_POINT);
  ...
  return false;
}

 ̀``


Hello!
Go here:
https://github.com/InTechStudios/The-Clash-Of-Kingdoms

Our website is here:
http://intechstudios.github.io/The-Clash-Of-Kingdoms/

Our organization is here:
https://github.com/InTechStudios

I will make you an owner later... I am too busy right now. Get used to Github, we can start the production of the game once I am back.


not impacting acceptance testing


Hahaha, good catch :) I will move this to the form class as well and check for multilang installations.


force push してコメントが消えてしまったので、メモ
https://github.com/TasukuNakano/perfect-ruby-on-rails/commit/7ff4560a983a51d80b0b9449ce1ddbf1d1bc34f8#commitcomment-16642762


'' doesn't  required here, you can use simple {SECOND_PASS:-}


dropbox?


expect の引数をブロックとして渡すことで、そのブロックの処理の前と後の動きでどのように変化があったかというテストをしています。


all properties has to have "private" modifier
Values may be set in constructors, or in setters.
To read properties use getters


:+1: a good commit msg for github,`Save space a little bit `


Note that if, at a future point you want to build documentation for dcore (like the firedrake API docs, say), you can give these attributes comments that will be interpreted as docstrings by sphinx like so:

```python
class Conf(Configuration):

    #: This is documentation for the attribute immediately after
    #: this comment, which may span multiple lines, but must
    #: begin the line with "#: ".
    #: The gas constant for dry air [J/kg/K]
    R_d = 287.0
```

See http://www.sphinx-doc.org/en/stable/ext/autodoc.html#directive-autoattribute for alternative syntaxes.