labofsahil/github-event-dataset-2012 huggingface dataset:
Пришлось пока отказаться от обработки буквы "ё", т.к. не понимаю причин ошибок при её обработке.
I am running into a dead end [here](https://github.com/SpoutDev/Vanilla/blame/9138a77f5ca4bcf851277369b0556e519a1480fd/src/main/java/org/getspout/vanilla/mobs/EntityID.java), so I have no idea..
Hi Fabian,

In fact, I didn't do a pull request for that """"feature"""", as I didn't think it could be useful somehow (and especially after you fix for old IE).
Generally, when I think something (feature, bugfix,...) could interest you, and if it is finished, I create a pull request.


> Did not merged it directly, but over the other branch 
"contrib-integration" and then merged with the master (for test them 
first).
> Hmm the graphs looks not nice now looks like you have to make
 pull requests to the contrib-integration branch, then it will 

> perhaps 
look better in the graphs. Is this possible ? I thought that it will 
recognize the flow Angelo and me defined, but it looks not...
Concerning you problem with the graphs, I had a look at the pull requests, and it seems that I cannot change the destination directly. The only solution is to close and recreate the pull requests. Note also that in github the destination of a pull request is by default the "master" branch. It's also a bit strange to do the requests on a branch which is not the master one, but why not.

Do you want me to recreate the 4 last pull requests? Or do you want me to use from now the "contrib-integration" branch?


Cheers,
Yannick




>________________________________
> De: Fabian Gander <reply@reply.github.com>
>: Yannick Charton <tontonitch-pro@yahoo.fr> 
>Envoy le : Dimanche 5 fvrier 2012 19h04
>Objet: Re: [NConf] added a warning if the client browser is not supported by NConf (16704d5)
> 
>Good, I will ignore that. Thanks anyway for the contrib ;)
>
>--- 
>Reply to this email directly or view it on GitHub:
>https://github.com/Tontonitch/NConf/commit/16704d540150f30d5a44f235874537e9447dc051#commitcomment-934505
>
>
>
Good catch--I was porting this from our internal site_xxx.py set, and I missed some imports.
Oh you are. Looks good then.
I think this is a little wrong.

In case I dont have Ncrunch env variable set, the condition Ncrunch != 1 will always be false, so tests never run. I have to manually set it to 0 if I don't use NCrunch.. I'm not sure people be doing that )

Some hours ago I did a pull request with similar fix, please check it out.
I do too. despora.de too afaik.
@luislavena: So I would simply replace 'i386-mswin32' by 'x86-mingw32'? Or can I target multiple platforms at once (64 bit, too). I followed the description in rake-compiler's README:

>To target gems for MRI Ruby’s current official distribution, please force the platform to the one (i386-mswin32) previously shown.

Is this outdated because RubyInstaller uses x86-mingw32?

How would I proceed if I wanted to try the gem out e.g. under Windows 7?

- First install Ruby using the RubyInstaller. (Let's say I choose 1.9.3)
- Then let's say I package krypt-core as a gem and put it on my Windows machine.
- Now what would I have to do so that 'gem install krypt-core-xxx.gem' succeeds?

I understood that in the general case (not relying on DevKit) I would have to provide
a 'fat gem', no?

Could you please explain that a bit further to me, or better even, point me to some project
where it's already done?

Thanks in advance,
Martin
should this be localized?
Looks fantastic otherwise. Thanks for updating the this file.
Sorry, I screwed up and didn't merge this fix in time into the master repository. The same issue has been resolved in version 1.0 though. Thanks for the fix regardless, I will try to keep a closer eye on forks in the future.
Good catch! :-P
Je trouve le rendu parfait. Mêmes commentaires que Binbin plus :
4) Il y a deux liens "équipe"
5) Prévoir une place pour une pub 468*60 en bas de chaque page sauf l'accueil (désolé de ne pas y avoir pensé plus tôt ^^)
@grum 
Pro testing being done on bleeding before pull requests are accepted! This causes insta-crash on client. See:
https://github.com/Afforess/CraftBukkitPlusPlus/commit/dbaf462b97c03a794dfb3b531510cc8d8b854847
Well, obviously I should not have started messing with branches. Crap.
shouldn't this line have one more " ?
I still don't understand what

```javascript
(trimmed.substring(0, 1) === '<' || trimmed.substring(0, 4) === comment_mark)
```
does. :)

New code accepts any string like "< ...". And (automatically) any string like "<!-- ...".
No worries and no way for you to know - yet ;-)
We can probably simplify this since you added logic to let the connection accept a promise for a port.
Create a new method for finding words which are a translation for the word
hmm - I did that so we had full support for the worldcat-based library translators.  http://melvyl.worldcat.org/ etc.
Suggestions on how to proceed on that front? Just create another Wordcat translator that doesn't double as a search translator?
`for (var key in Array.prototype)` doesn't do anything in other browsers either, because those methods are not enumerable (per ES5 spec).
Thanks. BTW, it seems on pentadactyl nightly build, `"use strict";` should be commended out. And TabGroupie does not play well with TabGroups Manager(https://addons.mozilla.org/en-US/firefox/addon/tabgroups-manager/), tgroup-* commands don't work.
Etrange, en effet. Je ne pense pas que time.sleep() puisse avoir un problème de résolution...
La doc en ligne indique "The argument may be a floating point number for subsecond precision."
do you really mean `print()`? I bet you actually mean `message()` or `warning()`
田丰的tiny错了
早加了，你拉了最新源码就知道了
instead of what?
Why not follow the same syntax as with form mappings?

```scala
implicit val carFormat: Format[Car] = productFormat2("id", "models")(Car.apply _)(Car.unapply _)
```
why are you addslashes?
BTW: For changes like this one, I recommend to use a guard block like

```java
if (!condition) {
   return defaultResult
}
```

instead of nesting ever more if-conditions into each other. That is more readable on the long run, in the code as well as in the diffs.
this is the only line in the whole file using '==' for test.
for consistency please replace with '==='.
This could reference `$pagename` right?
Yeah, sorry about the tabs.  The problem is that Solaris/ON rules require hard tabs (which I've always hated).  I have MIT vim settings but I keep forgetting to use them when I fire up vim (the default C settings are for Solaris).  I'm thinking about using an env. variable to get vim to use a particular C editing config.  When I'm working on MIT code I'd set it to pull in those C format settings.
Great. And don't worry about the chromium issue, I guess that was just an effect of what you fixed in 58b8323882df6bfa4c51b9aea668265fc81a6235. I can't reproduce it anymore.
*whistles* Nothing to see here!
So the eclipse code formater does not break the fluent on a Ctrl-Shift-F
The new design hopefully looks a little better, and doesn't require style definitions for every different screen size, just landscape and portrait, plus eventually tablet-specific layouts if I get around to it. Here's some comparison screenshots.

Landscape
![landscape comparison](http://i.imgur.com/3NkGM.png)

Portrait
![portrait comparison](http://i.imgur.com/PONRa.png)

And tablet. Obviously it's not ideal, but it does demonstrate that the automatic layout scaling.
![tablet comparison](http://i.imgur.com/nejzU.png)
I forgot to add the form.py. Added in the development branch!
А зачем она такая нужна дура, когда есть нативная поддержка!

Я гонял тестировал скорость работы в 100-и раз медленне
Extract as default size double.
for server-side Eye of Acherus isn't moving
thic commit broken buffs and casts?
Hi Muki,

thanks for letting me know about your project change. Sorry that I didn't answer your email sooner.

The license agreements here do not target Knowing at all. It is for other projects (trove4j, javatools, stanfordnlp, weka,...); though it might be overlapped the objectives of Knowing. I believe porting Knowing to Maven does not affect the agreements.

Cheers,
Tuan
right, merged!
I think this is backwards.  y_direction=1 is used to go in the positive direction. Should be 

    dda->y_direction = (target->Y >= 0)?1:0;

Right?
Good catch!  :-)
ofc it is working (old spell coding methods) http://oi44.tinypic.com/3525j4o.jpg
but in light of this fix, its shown another flaw in this design,
this should not proc if you do not have the spell arcane missiles (http://www.wowhead.com/spell=5143)
 brand new toon at lvl 1 ( only start spells) http://oi39.tinypic.com/kbd5om.jpg
Looks like there's a typo here
ArticleView is also missing...
As discussed on IRC, this is a project dependency (not specific to pages/desktop) so should be a the top level.
Although it's not a harmful issue, I think it's better to place <sound></sound> outside <nodes></nodes> when it's not attached to a node. This makes it unified with the camera and the light. Please modify it.
I see I still have a `require_relative` statement left in the file, so please remove that if you're pulling this into master. Just a heads up. :)
Alliance Depot 是干啥的，到现在没明白。 :em06
... i tutaj dać `String text = ...`
I'm not on Widows right now, but it looks like this one might do the trick: http://www.paehl.com/open_source/?download=curl_724_0_rtmp_ssh2_ssl_sspi.zip
Usuniecie dziwnego słowa.
Great - I'll rebase to incorporate your feedback and send you a new pull
request - give me a few days.

On 29 January 2012 21:37, Philipp Meier <
reply@reply.github.com
> wrote:

> Look fine in general and I'dl likely merge the commit. Please remove the
> historic / anecdotical comments and double check the compatibility of
> licenses. Thanks for you effort!
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/malcolmsparks/compojure-rest/commit/8c0018034db7c15e8520caa5c76de5949c911adc#commitcomment-912588
>
shouldn't it be mozo?
Will this work for a blog with thousands of posts?
maybe it misses an `end` here.
I recommend returning content that matches the requested response content type.
Updating with South does not update my auth_permissions table. I think this needs a custom migration per http://stackoverflow.com/questions/1742021/adding-new-custom-permissions-in-django
You can merge this line with the one above.
How did this work? This should be a collection key, not the Sync Key.
I guess this is for connection stats in Ice Cream Sandwich?
Although `self.send(:association, association.name).association_scope` and `self.send(association.name)` are functionally equivalent, the latter relies on delegation to yield a scope in the `only_deleted_inside_window_if_time_paranoid`.  Passing the scope directly with `association_scope` seems more clear by virtue of being more explicit.
Mentions the requirements now.
Yeah, I realized that one the way home. I always forget to do that
On Jan 1, 2012 11:36 AM, "xoomdev" <
reply@reply.github.com>
wrote:

> Need back slashes on line 35 and 36
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/RootzBoat/android_vendor_rootz/commit/16834bb53fba7896c3868ad1475bbd23e584b146#commitcomment-830428
>
Okay yeah makes sense.
So long as that builds in the docs fine that sounds good.
Also re naming, I think it should probably move to CHANGELOG.rst.
I think removing this should do the trick for charging. But It will prevent power off from recovery from working probably, I think it will then boot into recovery the next time you boot up the kernel. I am working on a proper user space solution for this. I am thinking of some little user space program that can be invoked with the recovery post-boot and pre-reboot scripts. 
In general the param partition controls this behavior, and the param partition is just a flash dumped c struct with some bits in it.
Yes, I see now... I'm looking intoit.
How about ".init" which already caused a lot of grief?
The imaginary parts of the bath response function are included in the prefactors and frequencies, so that only one set of p and Omega vectors needs to be used.
agree. I'll change them.
yeah ... is true .. I make the change and send a new pull-request
`if (event.button === 3)` has the same meaning, so this can be shorter.
should it just silently swallow messages?
@drewhk this has the advantage of being O(1) when sending, but I agree that it's too memory hungry. What do you think about using something similar to what we used in `ConsistentHash`? An array of same size as the original routees Vector and with the accumulated weights as elements. Then using `binarySearch` to find the right bucket.
`bothOld == bothNew` should always hold, no?
If we aren't making this an FSM Actor now, I think we should also omit the
entire "state" thing, I don't like partial/ad-hoc solutions. Do you agree?
True, then I see no reason for it
Document as internal?
Some options: 
 1. I don't keep the messages when the producer is not ready yet, they just disappear.
 2. I throw an ActivationException when the producer receives a message and is not ready yet.
 3. I mess up the ordering ;-)
 4. like three, but without messing it up, how about if I call produce directly here and pass in the originalSender from the messages. 
Let me know your favorite of these 4
Will a Java programmer be able to relate to Scala Collections?
doesn't `foreach` look better than `map` for this?
Good to know. Why not everywhere?

/Patrik

5 okt 2012 kl. 20:24 skrev Roland Kuhn <notifications@github.com>:

> In akka-docs/rst/java/code/docs/pattern/SchedulerPatternTest.scala:
> 
> > @@ -0,0 +1,9 @@
> > +/**
> > + * Copyright (C) 2009-2012 Typesafe Inc. <http://www.typesafe.com>
> > + */
> > +
> > +package docs.pattern
> > +
> > +import org.scalatest.junit.JUnitSuite
> > +
> > +class SchedulerPatternTest extends SchedulerPatternTestBase with JUnitSuite
> I mean, only within the akka-docs and akka-contrib projects.
> 
> —
> Reply to this email directly or view it on GitHub.
>
I usually prefer to do a try/catch so its optional. 

--------------------------------
Sent from my iPhone
Chris Moyer

On Dec 3, 2012, at 5:50 PM, Mitch Garnaat <notifications@github.com> wrote:

> In boto/provider.py:
> 
> > @@ -246,6 +246,11 @@ def get_credentials(self, access_key=None, secret_key=None):
> >              self.secret_key = os.environ[secret_key_name.upper()]
> >          elif config.has_option('Credentials', secret_key_name):
> >              self.secret_key = config.get('Credentials', secret_key_name)
> > +        elif config.has_option('Credentials', 'keyring'):
> > +            keyring_name = config.get('Credentials', 'keyring')
> > +            import keyring
> I think we should either add keyring as a dependency in setup.py or put a try/except block here to give the user a readable warning message. Thoughts?
> 
> —
> Reply to this email directly or view it on GitHub.
>
Do we have any tools for pluralization? Or something simple will do? Like this:

```ruby
raise Sass::SyntaxError.new("Function #{@name} doesn't have an argument#{'s' if unknown.args.length > 1}: #{unknwon_args.map{|name| "$#{name}"}}")
```

Also it looks, like I've forgot to join unknown args into string…
One last change, can you change "orphans" both here and above to say "orphaned units"? You use the term below in the all flag and that one reads better.
This should probably be a ValueError. I think you would only raise TypeError if the type was not a string. http://docs.python.org/2/library/exceptions.html#exceptions.ValueError
`included`
True, as we go back to one task per feeder feeder. Will remove ASAP.
same here...seems superfluous
Good idea.  Not sure if it can be done without using the idShow at all, i.e. returning something like

negate + "(watchedcount > 0 AND watchedcount < totalCounts)";
Sure. It's fine to extend it. But eventually we should probably pass the whole packet.
I'll remove this ifdef.
Moreover, there are no "disable_winjoystick" on other platforms.
secound -> second
Question: Is there a default datasource already configured in Weblogic 12c ? That would be much easier to quickstart.
Given that the *.st files *should* be contained in the resulting proto-singletemplate JAR, I'm not going to merge this hunk.
Should this class not be merged into CascadeNetworkExpansionReactionStrategy? Then there would also be no need for MultiReactionStrategy. Although multiple reactions could potentially be useful. Comments?
This implementation is wrong. It will return false for existent templates if you have a non-advanced loader in the LoaderChain.
For non-advanced ones, you have to try to load the template and catch the Twig_Error_Loader to know that there is no template
`super` needs to be invoked within this method.
Just setting the timer to nil isn't enough, it should be invalidated as well.
This should account for the case of a major version greater than 10.
Use one or zero blank lines here, not two.
Exposing IntPtr in the public API doesn't seem to be a good fit for C#.  The Java bindings use the $0 tag as a handle in the public API and manage pointers internally.  I think the same approach would be good here.
Good catch. We tried to clean up all the string error syntax, but obviously we missed some.
перед звездочками пробелы
I think `dup` is not required in this class, is it?
What is the point of this test?  I don't like it because setting the instance variable makes assumptions about how the connection is stored and retrieved from the class.

Can you tell us what the goal of this test is?  Maybe we can come up with a different way to test.
The if condition can be removed as we're creating a separate `assert_redirects` method.
hahaha. Rollback.
I think we should mention the default value of `Date.beginning_of_week`
Unecessary line here.
Might be more succinctly described as:

    val p = newPromise[U]
    p completeWith this
    p completeWith that
    p

(Although it will incure 1 extra allocation, I think it's worth it)
Shape is a concept from the language spec. It's basically "approximate type". I am not able to  defend that checking the shape of these arguments does anything if you are about to check their  expected types right after, but it is part of the spec.
I had a fix in 2.10 somewhere that changes this one to be saclac.args.quick, as it stands it's kind of bad.
You could add to this description the range of frequencies that is returned.
I just noticed that this was removed by 75c8d2d65085895c30be515a99c2413e311df873.  Glad to see you've added it back (it used to be `return a / b, self.zero`).
Yes, None needs to be handled well, too. But that's on people's radar. We're going to get strange bugs unless the assumptions canonicalize the assumption values...just sayin'.
What about this:

```
us = UnitSystem_whatever_whatever()
us.some_method_that_does_whaetever()
x = us.some_attribute_that_I_will_use_often
```
I am changing ``metric_sym`` to ``metric_antisym``; if it is True the metric is antisymmetric, if False it is symmetric,
if None there is no metric.
Similarly `commuting` to `anticommuting` in `TensorType.__call__`, `sym` to `antisym` in get_symmetric_group_sgs
But this only works for `ImmutableMatrix` not for any other class derived from `MatrixExpr`.
Well, I believe that the default pretty printing an example of which I showed in [this comment](https://github.com/sympy/sympy/pull/1338#discussion_r969376) is quite OK.  As for LaTeX pretty printing, I'm not sure to what extent IPython notebooks support LaTeX and its packages.  It would be great if the full package set is supported, but if it isn't, I don't think implementing some custom drawing is worth it.
Maybe if something depends on the Bot configuration in that way but is general-purpose, it should be in the Bot class?  The only other alternative would be some kind of globals, which is ugly...
if (Log.isDebugEnabled)

otherwise we pay the concatenation cost regardless
Fixed in force-push
Needs no-op win32 implementations, otherwise LGTM.
i don't like these lines
actually, why is this diff old?
I don't have that typo....
so
project.default_for_user
?
the parenthetical needs to do one of the following: 

- be a part of the sentence.

- be in a footnote

- not exist.
I think that UUID should be always first, as its the most specific (and probably faster) lookup?
I think if I get it, this catching special situation because it may return null. Can you document why?
After closer review, this statement should rather be `_sql($sql, $errored, $error_ary);`
Is this no longer needed? If it is, uncomment it, otherwise remove the comment, along with the comment on the previous line.
Perform str_replace for $sequence in sql. This will become roughly:

`column_default = 'nextval(' || replace($table, '_seq', '') || '::regclass)'`
As this is against develop, `var` shouldn't be used. Also missing access modifiers everywhere.
That behavior then should be documented in the docblock of the method.   

--  
Erik Frèrejean
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Friday, July 6, 2012 at 14:09, Joas Schilling wrote:

> > + //array('msg_id' => 3),
> > + //array('msg_id' => 4),
> > + //array('msg_id' => 5),
> > + ),
> > + array(
> > + //array('msg_id' => 1, 'user_id' => 2),
> > + array('msg_id' => 1, 'user_id' => 3),
> > + array('msg_id' => 1, 'user_id' => 4),
> > + //array('msg_id' => 2, 'user_id' => 2),
> > + //array('msg_id' => 2, 'user_id' => 4),
> > + //array('msg_id' => 4, 'user_id' => 3),
> > + //array('msg_id' => 3, 'user_id' => 2),
> > + //array('msg_id' => 4, 'user_id' => 3),
> > + //array('msg_id' => 5, 'user_id' => 2),
> > + //array('msg_id' => 5, 'user_id' => 3),
> > + //array('msg_id' => 5, 'user_id' => 4),
> >  
>  
>  
> I commented the lines out that are deleted by the function and do not appear in the result, so you can see easily which elements should be deleted and which are still there
>  
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/phpbb/phpbb3/pull/874/files#r1112278
>  
>
>>You mean you want to rename the $row variable throughout the mcp_forum_view method?

>Strictly speaking this is not necessary. You can create additional variable(s) for the event dispatch.

IMHO it would be far better to clean up the private api rather than hack your way around that just in order to expose "pretty" names. But then again that is a different RFC/PR
doesn't this need a `if (latest)` around it?  I think on an empty directory this would be sprintf'ing in NULL
If we encounter the old format (or any corrupted file) we need to bail here, right?
This should be pulled off the prs readable attribute.
No, since we still need to distinguish between displaying an fps counter (`fpsLog === true`) and calling a callback with the framerate each second (`typeof fpsLog === 'function'`)
Guess we can have a team chat about this syntax. Never liked it myself, probably because older parsers would give you a semi-colonoscopy rather than wait to see if the next line started with a '.';
Это выглядит ужасно
Agreed. It's unfortunate that unicode strings get an ugly 'u' if we use repr (#27).
Lazy way is sometimes the best way lol.
How about writing a test that calls subprocess to execute this?
Spelling: lease should be least.
Also, grammar: we usually say "the test passed (or failed)" or "the test has passed or failed" rather than "the test is passed or failed".

BTW, I usually reserve spelling and grammar commentary for global symbols and text that is in some sense "user visible".

Also, please review all print statements and consider replacing abbreviations and (worse) internal variable names like logFileS with user-friendly text:
"Execute command" for "Exec CMD", "command line" for "cmdline", "Directory" for "Dir", "server log file" for "logFileS", etc.

... UNLESS your intent is to make the output more distinctive for the purposes of enabling a later pattern match -- but in that case I would expect to
see at least a comment and better yet a specifically called out pattern variable, for example:

    # This pattern is used to drive post-processing functions. Do not change it without a corresponding change there.
    error_pattern_formatter = "ERR: %s CMD: %s\n" 
    print(error_pattern_formatter % (short_msg, cmd))
General question -- why waste time adding "diff" messages to messages and then filtering and copying into diffMessages.
In the case of messages that do not contain "diff" or "Diff", why add them to messages just to throw them away?
Why not just add directly to diffMessages?
Weird that there's a throwDynamicSQLException as well as throw SQLException(SQLException::dynamic_sql_error...
models.DateField(default=datetime.date.today) or you will have to reset it everyday to change the date
I'll make a PR for `riak_api` and the other apps that adds a `deregister` function, and then calls it in the `stop` application callback.
этот `filter` вообще можно не делать, можно просто сразу `iterateJson(obj, ...`, на простых значениях из block, elem, mod и elemMods не наэкономишь -- у меня первоначально опасения были просто за левый контент кастомных данных, который в принципе в больших количествах может лежать в БЭМ-сущностях
Isn't this redundant since Model.h is included within FacilityModel.h?
Would be nice. Couldn't seem to get it to work.

It looks like it's been set up to link to the python docs in the conf.py but `:ref:datetime.date<python:anything-here>` (with the necessary back ticks) was giving me an error:

WARNING: undefined label: python:datetime (if the link has no caption the label must precede a section header)
I agree. One of the things that confused me while learning Django was repeated references to URLconf when what I really needed to locate was urls.py
I'm throwing my notes here, so that you can see how far I've gone. 

- Read some SSH code and there is definitely things lying around the Transport object that could help us knowing the connection is up or not, although Google does not know.
- I've decided (because I have no network cable easily available), to try to simulate the disconnection when using a simple SSH tunnel that I'd abruptly cut.
- Running ```demo_simple.py``` (from the ssh package) and shutting down the tunnel sends en EOF 
- Running ```fab -H localhost sleep``` and doing the same does this:

```
fab -H localhost sleep
[localhost] Executing task 'sleep'
[localhost] run: sleep 30

Fatal error: run() received nonzero return code -1 while executing!

Requested: sleep 30
Executed: /bin/bash -l -c "sleep 30"

Aborting.
Disconnecting from localhost:1212... done.
```

This looks good. Now I will try to emulate the "unplug network connector" and run the same tests (demo_simple is cool because it's small and its behaviour will reveal how to fix the issue in fabric). Not before an hour or two though because I'm not having enough machines around to do the test properly...

(as a side node, I've read a few things about TCP retransmission and the usual kernel timeout seems to be around 15 minutes http://stackoverflow.com/questions/5907527/application-control-of-tcp-retransmission-on-linux ) before the kernel decides the connection is actually cut. This value I'll have to check against the unplug-cable test...
Here you iterate over mesh point sequentially. You don't have to slice the mesh if you restructure the loop a little:

```python
for i, location in enumerate(points):
    poes = hc_progress.result_matrix[i]
    # ``location`` is the point object corresponding to ``poes``
...
```
Can you move the timing after the `:` so the output is still parseable by python's `config` module? Something like `<name>: <status> (<time>)` perhaps
Same here: `toSelector( tokens )`.
The indentation here is ambiguous.  I assume this line is not inside "if" block but should be run always.
@fschopp What's your opinion?
This is a good catch.  We have corrected.
If you think EU should be doing this, make sure to create an issue for it.
I needed it for some reason.
This one. This shows that you love your users. :)
I just thought it would be better if I avoided putting another inline search, therefore I factored out the user searcher from checkPassword...
Other tests in the same file are already using strings and nothing in Vumi *tests* is using classes so I'd like to keep the consistency.
missing `yield`
I think some transparency about what people are working on should only be neccesary for bugs, as there is a much higher chance of overlapping work here than anywhere else. Like you said, errors are something we should be concerned with.
No, in the Git processes the files in the course of commit, the content of the file is zipped in a blob which is indexed into the database under its SHA-1, then the commit content is built to point on the SHA-1 of the file. the commit content is hashed and zipped in a blob which is stored in the database

With this order of processing, It is clear that you cannot modify the file with information from the commit, because it would change its content, thus its SHA-1 and the commit object would not point to this new content.

Maybe to be clearer, the phrase could be "you can't modify a file already inserted in Git database with information about the commit after you've created the commit" if it still makes sense.
Fixed. Please check the second commit
Want me to change it?  There are a few different ways to get a file into a string.
I don't like unitary much, except compared against Set binding.  createSingleBinding?  createSoloBinding?  createValueBinding?
Given this has been rewritten, worthwhile trying to resolve GPII-1 by updating to latest Infusion trunk and eliminating this function and set of comments entirely.
Actually, it's ok if we're returning the original path here. Notice in `MainWindowViewModel.cs` [line 53](https://github.com/davidanthoff/SeeGit/blob/a4438ec084a50e18f8423cfd49309a206b76d11c/SeeGitApp/ViewModels/MainWindowViewModel.cs#L53) if the directory you give us is not a .git directory and doesn't have a .git directory, we monitor the directory waiting for you to create one in there.

In my demos, I start monitoring an empty directory. Then I create a git repo in there and add commits and SeeGit automatically comes alive showing the commit graph. :)
No, it won't. The "Stacktrace was: " prefix is generated inside `StacktracePrintingMatcher`. I didn't want to check for the complete stacktrace that comes after that line because it contains line numbers and class names that will change in the future. :-/
Is there anything that we should expect to see in the file that's created that we can test for?
formatting nitpick: should be a space between `if` and `(`
Yoko says it's ok to drop country code since the localization team doesn't differentiate french regions. Also, see my prior comment about how RequireJS handles the hierarchy.
For the record: Glenn and I talked about this a bit on IM and settled on leaving it as-is but documenting the limitation/assumption more clearly.  In some common cases where the mode is mixed but starts & ends the same, it's actually ok to just use that mode (e.g. a selection of HTML that includes a script block somewhere in the middle of the selection can still be commented out).  If bugs from the cases where it's not ok become a big deal, we can revisit and enhance this later.
I'm feeling nostalgic. I'll say PopUp. :)
I think we should just take this out. There's no real harm in starting a new load while the old one is still pending--it just means that they'll both resolve and `onCursorActivity()` will be called twice. I'd rather that happen than have an exception be thrown by the assert, which might lead to other state being messed up since we're probably not careful about exceptions in callers of this function. (In general, we should probably avoid `console.assert()` altogether unless we're in a really bad situation if the assertion fails, since throwing the exception can lead to worse problems--we should just log instead.)
Can't you just put a simple call to "SpecRunnerUtils.closeTestWindow();" in afterEach() ? Or, is this different?
It's maybe not word by word, but reflects it in a meaningful way, doesn't it?
``reporterView`` is unused.
you need to copy more than just the doc
e.type and e.origin are needed as well

See: https://github.com/ajaxorg/cloud9/blob/8c6b2e60d3a2605cc191350b4a08188b2caca611/plugins-client/ext.filesystem/filesystem.js#L497

also, why is this event not called from within openEditor?
Why did you change this? It makes the loop slower...
On mac systems, glob returns file in alphabetical order, but this is not the case on linux.  It might be a good idea to systematically sort the filenames after a glob call.
looks like we don't need this `to_i` here. It was `to_i`ed at https://github.com/defunkt/resque/pull/671/files#L4R46, right?
note to self: addressed in updates.
I don't see any glitch/flicker with the Nexus 7 but perhaps different hardware will behave differently. Removing the "setContentView" didn't break anything, so it is better that way.
I don't see any reason why not - but do you think other packages will have a use for it?
This is the token used for variables in doc-source-url by scaladoc, see `scaladoc -help` for more info.
Note that I know this should not happen. But "should" does not give any certitude. The other way to do that properly would be to throw NPE for null streamSource...
in editor helper on line 38, we seem to set the innerHTML to something different than this. perhaps we can make it more consistent?
Can you declare trackEvents and tracks on line 148 instead of inside the for loop
Ah, I see it now.

Very good, sir.
Since the functionality of `closeEditor` is warping pretty drastically from what someone would expect it to do given the simple name `closeEditor`, let's rename it to something more appropriate. Perhaps `closeTrackEventEditor` or something more concise.
fwiw, I'd also be in favor of *helping* them to build a manifest somehow, but that's not necessarily in scope for mortar, we might just point them to a manifest builder somewhere.
This code is also running in the pdf.js add-on and we are trying to support Firefox 10ESR and other things that are using Gecko (e.g. Seamonkey) for now. We will be keeping the try/catch-es -- I added related comments in case if somebody decide to cleanup later.
Move !pdfTitle to the first spot so if there was one already we don't have to check info for one because of short circuiting.
I often have this discussion amongst myself.  I would usually draw the line at [...] being less than about 30 characters long, i.e. a pretty short thing.  Half the time when I start with `return [...]` I end up changing it anyway because I want to debug or view what's in the list before returning, so these days I tend to just make the one-off variable most of the time.
Please, for future, leave this kind of changes to "white-space" commits, it makes it easier to track history changes, especially when something goes wrong on this line.
No, it's not hanging at all for me... test passes perfectly.
`https` instead of `http` here.
Revert this change per previous comment.

I am sorry that we are being English-ignorant.
(I am surprised by myself to have to say this, as an Asian)
For the record, the Keyboard will be a separate application so it does not really need to have a Gaia.UI namespace since it will live into it's own scope.
the whole `/tmp/rt` won't exist in X environment is the reason we have `rt_scratch_dir`, although, putting it under the git repo which defaults to `/tmp/rt` would guarantee it's deletion between tests. Any reason why we couldn't have the post commit hook send a message to the test's process and set up a receive block?
is the `\` needed?
no idea what Contestualize means, but please avoid inline comments. if it's not in a doc block - it doesn't show up in the api.
I propose we bail on sending `X-Couch-RYW` headers for `sorted=false` views.  We can only do a crappy job of it anyway, since  we start the real response as soon as we receive a row from any shard.
ಠ_ಠ I don't care for this type. Maybe another page dedicated to them?
Ok, this whole code needs to be refactored.
It breaks the rule #1 of Object Calisthenics: "Only one level of indentation per method".
You can rewrite this one using early returns

```php
private function getNamespaceVersion()
{
    if (null !== $this->namespaceVersion) {
        return $this->namespaceVersion;
    }

    $namespaceCacheKey = $this->getNamespaceCacheKey();
    $namespaceVersion  = $this->doFetch($namespaceCacheKey);

    if (false === $namespaceVersion) {
        $namespaceVersion = 1;

        $this->doSave($namespaceCacheKey, $namespaceVersion);
    }

    $this->namespaceVersion = $namespaceVersion;

    return $this->namespaceVersion;
}
```
"wishes for their post to be visible"
status (to parallel the industry job status)
Unusual hashCode implementation. Why not classic one:
val prime = 31
var result = 1
var result = prime * result + if (foo == null) 0 else foo.hashCode()

I see that's how JDK HttpCookie is implemented, but still?!
This line should use the JQPivot namespace constant.
I think `get_default_git_binary` should throw an error if it can't find one
Yes I am.  The js can call `e.show()` if it wants to draw.  But it is *certainly* true that if there is nothing to draw, the height should be zero (right now it's ~15 px, so running a series of js calls results in infinitely growing vertical space).
Yes, self.exit(1) works great.
The getTitle and getDir functions should probably go away.  They're very simple and just appear to pass through based on a check of the file extension.  This logic can be confusing, e.g. when is the title "stdin" ever going to be useful in an output file, or why is the output value "stdout" or "stderr" ever useful?

For example, the getTitle function is made obsolete by my suggested title statement change above.
`#define GIT_REMOTE_DEFAULT_FETCH (const char *)-1`? :trollface:
Share function header! Move macros inside!
This error code is going away. If you need to store errors inside the transport, you may want to change it to `t->failed = true`.
No return?
We've got a @baseLine Less variable for this - does the 24px not work?
I am also fine making dest a required parameter and having to specify the particular app you care about
Really? 123.1%???
"advanced"
also, @wraithan has a nifty script for this that he's about to commit :D
`stylus` throws a `RuntimeError` when the runtime isn't available, so I think would be better to rescue for just `RuntimeError` instead of `Exception` here, so it won't swallow any other kind of setup issue that might appear.
@fabpot what's your input on this. I know I'm not doing something pretty here but there's no standard way of doing this and I can't think of a better way of doing it. Should I rewrite this and make it better but possibly break the current custom panels or leave it like this and have the other panels adapt to the default ones.
Please add this as a use above and refer to the alias.
This condition will never occur because PHP will only open one session at a time.
ok, will change to `/`
If provided string is not a UTF-8 string, `preg_replace` should throw an error and return `null`.
please rephrase to 

    added method `guessPattern` to FormTypeGuesserInterface to guess which pattern to use in the HTML5 attribute "pattern"
That should be `integer`.
``if (!$this->isRunning()) {``
@petrbela, this looks a little weird for me, having the same function for all of the keys despite being able to specify individual functions for each key.  Did you base it on a different language file?

cc @timrwood
We can create a twitter/ambrose hackweek branch for now and avoid merging to master.
I didn't know there is a maven plugin. Let's add that to the sample maven
file too.
On Aug 29, 2012 8:30 AM, "Steve Gury" <notifications@github.com> wrote:

> In README.md:
>
> > @@ -182,6 +182,8 @@ Apache Thrift is a binary communication protocol that defines available methods
> >        string hi();
> >      }
> >
> > +To create a Finagle Thrift service, you must use the <a href="https://github.com/mariusaeriksen/thrift-0.5.0-finagle">custom Thrift compiler</a>. It will generate, in addition to the regular Thrift `Iface` interface, a `ServiceIface` interface that wraps all return values in a `Future`, which is required by Finagle.
>
> I would write something like that:
> """
> To create a Finagle Thrift service, you just need to implement an
> Interface generated for Finagle by Scrooge<https://github.com/twitter/scrooge>.
> Scrooge <https://github.com/twitter/scrooge> is Thrift code generator
> similar to the standard Thrift compiler <http://thrift.apache.org>, the
> main difference is that the Interface expose asynchronous methods
> compatible with Finagle.
>
>    - If you are using sbt to build your project, there is a plugin
>    sbt-scrooge <https://github.com/twitter/sbt-scrooge> that will
>    automatically compile IDL for you. Beware that Twitter has moved out from
>    sbt, so the latest release version of this plugin is only compatible with
>    sbt 0.11.2
>    - If you are using maven, there is a maven pluggin
>    "maven-finagle-thrift-plugin" that do the same thing. """
>
> What do you think?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/twitter/finagle/pull/105/files#r1486589>.
>
>
Gotta have tests for this since it's hard to deploy fixes to agents. [tests/test_datadog.py](https://github.com/DataDog/dd-agent/blob/master/tests/test_datadog.py) is a good example of passing in sample data and checking the result
Remove the ```Z```.
I think we already solved this issue in VanillaMaterials...can someone come along and confirm this for me.

@Olloth
In other repos/languages we have the convention of always finish the file with a new line, in case you want to adopt it
Should probably reference http://www.w3.org/International/O-HTTP-charset.en.php#charset that mentions how charsets are only expected on text/* mime types.
are you sure this is what we want? `0771` is octal, which is what we need here for the file permissions. this way you are setting completely different values.
I created https://issues.jboss.org/browse/AS7-5772 to further track this. Thanks
hmm, this conditional belongs in the  first arg to mandatory, I'll fix that.
That's what the `TBD` in line 260 is for! The rare case would be like the following:

1. User set the clock alarm
2. The alarm rings
3. User ignore it (but not dismiss it) by pressing home
4. User got a call
5. User answer the call
6. User dismiss the call screen by pressing home
7. User launches Clocks app from home screen

I will leave someone else to figure out how to implement this ...
Thanks for the fix; I had this change locally too.  My local change also updates the addrList.push's address field to use node.value.trim() too.  Unless you see a reason to avoid that, I think we want that here too.
is it synchronous?
On 19/06/2012 13:20, Cristian Rodriguez wrote:
>> @@ -2,7 +2,7 @@
>>   'use strict';
>>   
>>   const PaginationBar = (function() {
>> -  var style, percentage = '%';
>> +  var style, percentage = '%', pTotal;
> by the way now the homescreen doesn't receive the DOM_VK_HOME event but the DOM_VK_ESCAPE yes, do you know something about it? thanks Vivien

Yes this is because the DOM_VK_HOME is caught at the platform level and 
send directly to the system app instead of this one. The exact behavior 
of DOM_VK_HOME is still not 100% define and it needs some work at the 
platform level to decide what to do.

the main reason why it sounds bad to dispatch it to the content is 
because the content will then be able to prevent the default action of 
the device (in this case, show the homescreen). That would be resolve at 
some point.
reg.onerror = onerror() {
And by "Just Work", I mean that if code does

    window.open("foo.html", "name");
    window.open("bar.html", "name");

you should get exactly one mozbrowseropenwindow event, for the first one.  The second open call is entirely transparent to the embedder, or at least it should be.
the string encoding API is very close.  There are patches up for review on https://bugzilla.mozilla.org/show_bug.cgi?id=764234 and I got the bug marked blocking for e-mail.
Now I'm not retrieving the `widgetFrame` each time but I leave these two functions. I could write just one setter but then I would need two closures, one for showing the widget, and another to hiding it.
The difference is `edit_mode.css` is a whole component w/o states, but `lists.css` has the `edit` state. If we want to use `data-type="edit"` we should use a classname for replacing the isolation provided by `data-type="lists"`
nope, has to be removed
Adding a button does not add much value.

We need to make sure the receiving app can properly cancel the activity when the home button is pressed. We don't want the app that started the activity to be left hanging.

Thankfully the `mozvisibilitychanged` is properly triggered when you close the window so the receiving app can to the `postError` at this moment.
it works just fine, but i'd rather not store primary keys of things outside of solitude if i can, preferring uuid. this allows us to get and post directly on the resource and store the uuid in webpay, rather than do puts and posts on the list, using the uuid as filter. In fact perhaps I should move this down for use by more resource.
The changes from [key] to .get(key) are a behavioural change. I think .get(key) returning None is better than getting a KeyError though so I'm +1 on the change.
Why are you unshifting twice in a row? You `unshift` to add a directory to the load path, so to match other Ruby code in AppScale, you should `unshift`, then `require` everything in that directory, and repeat this process as needed. This also helps for the (hopefully) rare cases where two files have the same name but are in different directories.
This isn't a standard pydoc tag, so move it into the description above.
This is a bit messy. Consider refactoring `place_error_app` to take a hash, with keys `:app_name` and `:error_reason` to clean this up, throughout this commit.
&view=profile has been deprecated a long time ago. Only CKunenaLink was using it.
@drchoffnes -- correct me if I am wrong, but this should be already taken care of right?  We have removed the need for a google.com address, replaced it with the need for a gmail.com compatible address and it should build and deploy from an external GAE instance.
`> 0.4.6 < 0.8` will be better. Node can release some release in the future that will break chaplin etc. Happened to me many times.
Another minor style issue: `opt_func` -> `optFunc`.
the property is undefined (which makes the class slower on PHP 5.4 due to the new optimizations)
what if one package of the whitelist is explicitly locked in my composer.json too ?
> This means that if any of the dependencies get a new version, you won't get the updates automatically.
reference to "buttons_cfg.h" which no longer exists
Pattern may be assigned in constructor
it provides assets for raphael.js
Discussion around this suggests that it means there's not a dependency for `.js-off` to be there before we add `.js-on`.
why not use a std::vector<std::pair<std::string, std::string> > here?
Yeah :thumbsdown: to this change.  I'm tryna to see more uses of 'tryna' in my docs.
Isn't it the same function than require("api-utils/private-browsing/utils").setMode?
  https://github.com/erikvold/jetpack-sdk/blob/fa3121b8b2402a6d6d0a8e0be6360cb77aa8e982/packages/api-utils/lib/private-browsing/utils.js#L29-46
This one looks more correct than utils.js one because of previous comment about `privateWindow` being only a getter.
I didn't want to affect the the base margin styles for top/left/bottom. Is there a better way to do this?
and none of the else are necessary
We don't use the context blocks in MSpec. A describe block should be fine for cases like this too.
CS. Length.

Aim for 80, max 120.
However is hard to believe that in a INSERT statement the part of field names is equal to the part of field values
$db is already checked with instanceof
This is a boolean operation, so just call the method `escapeLabels`.
Oh, I see now..
I didn't notice the else if (func_num_args() > 1) check
what you stored in the property is **not** a markdown file currently (well, assuming your code is fixed). And you should use the config dumper to generate the proper yaml
Regarding Ordinal vs Invariant: http://blogs.msdn.com/b/michkap/archive/2004/12/29/344136.aspx

Primarily, it's about perf these days. :)