labofsahil/github-event-dataset-2012 huggingface dataset:
Do you really need to move the directory?
I know moving the directory is needed when you have a plugin, so you move the directory to the cloned cakephp directory. But if there is no need, I would just remove this line.
Just cosmetic changes except line 46 . I was getting null pointer exception in the navigation method so fixed by Payal
@christianv can you change this to mvn clean package jetty:run to match the instructions on the readme.md
Do we need for this to be globally available at all? I would advocate just doing the renderNav functions within the views that actually need left nav.
There is no performance benefit to calling an async ndb method and then immediately calling get_result, afaik.
good initiative. I wonder how easily we can get qdoc running over the whole plugins tree...
http://blogs.technet.com/b/grouppolicy/archive/2009/04/22/passwords-in-group-policy-preferences-updated.aspx
Did you intentionally remove this check instead of just changing the minimum value to -1?
Eh, using the regex library seems overly complicated - it'd have to be stubbed in somehow in ap_stubs.c for testing too.  I broke this out into a separate function and knowing that someone else is on board, included gotos :-)
lulz I mostly meant commas at all, but you'll use the Oxford one if you want to live. I mean... whoops.
> Why use a separate name? GLSL has function overloading.

I wasn't sure if it did, so I googled it, and the results led me to believe that it is supported but that it has a non-zero chance of being buggy.  So I didn't use it.  I'd be happy to use overloading here, though, if you feel it's safe.
Invert this to reduce nesting.  Similar applies to other DynamicTypes.
Hmm. OK.  I'll change it.
bdrip exists as the rip from release version of blue-ray disc, brrip is rip from prerelease blue-ray dics http://en.wikipedia.org/wiki/Pirated_movie_release_types
does that make sense conceptually? Should an error be thrown if the match failed?
it IS already "it's", you mean it should be "its"? Anyway, it should be "its", that was a typo there :)
Instead of this, how about just naming the file:

`navgroup.xml`:
Thank you - I have done that in my next update.
Note that you included a link as <Ti.UI.FILL>, which will not resolve. It must be <Titanium.UI.FILL> for the moment, although there may be a ticket open to fix this. I have fixed this in the next update too.
6 spaces of indentation is kind of weird.
Combining the IGNORE and TRANSLIT sounds like a plan. Throwing an exception is something I will refrain from because it may happen that you feed an empty php file (which would be silly) and technically this is not an error situation.
Move `licenses` before `developers`, and `modules` before `scm` (I'm not found of this last rule, but let's stick to the style guide)
Да это вообще не нужно, вроде бы. Это при дропе уже происходит в том страшном месте с кучей e e e.

Я проверю.
Why not add it now, release is done.
If you start putting comments like that in the docs, then pretty much _every_ range-based functions is going to need them. It's a given that a range-based function is likely to consume a range. It's just that the range types that are most typically used automatically save themselves when passed in. That being the case, it seems like extra clutter to me to put a note about it `walkLength`. And I really don't like the "_may_ be consumed" bit.  To me, hat makes it sound like it's questionable or nondeterminstic as to what will happen, which is not the case at all. It's entirely a question of range type, and anyone passing in a reference type range will know that you have to save it or they'll figure it out real quick, because it'll get consumed on them (the same will happen with most range-based functions).

It seems to me that you're putting this note here just because you just found some calls in Phobos where someone forgot to `save` the range before passing it to `walkLength`, and putting a not on `walkLength` really isn't going to help with that. The problem is that functions are frequently untested with reference types and people forget about their quirks. What's needed to fix that is better testing for reference types. You're putting a note on one function about something that's a general issue that needs to be known about and coded for. It would be like putting a note on a C/C++ function reminding people that C/C++ isn't garbage collected and that they need to make sure that they call `free` or `delete` when their down with something allocated on the heap.
mmm - I'd prefer messages that are more concrete to understand, without me having to know about other systems like AngularJS. How about:

* support for two-way bindings in radio buttons (issue #198)


Note: John hasn't submitted those changes here yet, so we should wait before we add these items in the change log.
Actually, why not just directly create and delete the account before and after the test code? (`setUp` and whatever the opposite is — `tearDown`?).
well, the ida is that the entire checking is optional. maybe we could use the config admin to validate this
What's the point of this declaration. Since it looks like you only use it once, why don't you just use $request->id.
are you proposing that there be no default?
we should back this with a non-JS per-drug view, definitely in a separate set of commits/pull requests
Good catch; the scope isn't really necessary here.
<a>test</a>
It still doesn't make sense to hard code the disabling of Chef in development.

The code should work more or less the same across all the standard Rails environments (development, test, production). This way we can run the different type of tests at the right level. For example, developers will naturally use the development environment to manually test features as they work on the code. The unit, RSpec, and BDD tests should use the test environment. And then the final integration tests (eg. mkcloud) on the production environment.

I suggest moving this to a configuration file with sane defaults. This allows devs/users to easily change the options without having to modify the code. For example, using http://github.com/railsjedi/rails_config allows the use of `#{environment}.local.yml` files to override the defaults. The `*.local.yml` should not be stored in Git, hence allowing devs to have their own settings without affecting the defaults stored in Git.
new TagImporter object handles this all now.  Added docstrings.  I called it "TagMap" at first, but realized that it's main function is the importer, not just mapping.
I happened upon some in the goals DOM which is why I had to add this. Think this class should be much more specific.
MINOR: Refers to .h not .cpp in comment.
Calling to set breadcrumbs from displayResults(). (page view)
Hmmm... this reads like the resource is transactional (which we know it is not).

Maybe...

    IntegrationResourceHolder

??

But I do agree it should not be MessageSourceResourceHolder (I actually noticed this while writing the docs - which will need an update for this when we decide on the name).
Same, make sure that all private fields are either 'volatile' or 'final'
I was thinking the same, but does the dns group exist if DNS isn't installed?
needs t()
This must be:

    'ADMIN_MEDIA_PREFIX': get_admin_static_url()

The same with the other repeat lines
ソート列名とソート順序のいずれかのみが null の場合(パラメータが)は 400 良いと思います。
ALL or Nothing がシンプルなので。

あと、列名や順序のデフォルト値も特に必要ないかと。
I was hoping to avoid all links to anything under support/{omero*,bio-formats} if possible.
I'm not taking all the credit for this. Put your name in here (first, last, not fussed).
what a jerk.
Usage of ` is sort of prohibited. @michaelficarra could answer better.
No, as far as I know it doesn't. Triggering all the events by ourselves seems safe + consistent.
Do you have some examples for service filter? I referred https://github.com/WindowsAzure/azure-sdk-tools-xplat/blob/master/lib/iaas/blobserviceex.js#L72 for this. I think we need to review the entire CLI tool for such usage and fix it to use service filter.
For sure.  I think we can parse `git config remote.origin.url` or use a ruby git library like grit or ruby-git.
same Problem a Server function call on client...
sure :D just pull the creation of AnswerSet out of that loop?
get_ancestor_or_self then?
move public methods to the top, private methods to the bottom. easier to navigate
Missing the variable name i.e. ````@param {...} type ...
Actually, Java implements unboxing of null by throwing a NullPointerException. I'm not sure which is more surprising.
Is it there because it needs to be online to make sense? I think if you can put in on people.opera.com it's a bit easier to test and doesn't need the ugly ssl.opera.com link.
It is better to add space after operator characters:
```javascript
this._padding = parseInt($(this.dialogID).css('padding-top')) + parseInt($(this.dialogID).css('padding-bottom'));
```
The logic here is a little strange. Having both a "cmdline" and "cmd" and both being the command-line module.
If you don't mind, i think we should change it. It's cleaner on one line.
please fix typo in method name
our ui.dialog JNext does it, too.
Should be counting Number column
It's a sample/test project, stop being Anal.
Is it also breacked in the 2.0 branch ? Or a bad merge ?
Succeeded already :O But either change it to false till succeeded or change the name
The wait is there because it takes some time for the file from creation to become fully populated. waitsFor will return the moment the file is created, but the file isn't completely written yet. If waiting for a fix amount doesn't look good, we'll have to look for another way to do it. Simply taking out the wait will not work.