labofsahil/github-event-dataset-2012 huggingface dataset:
With these, I shied away from ucfirst. In other packages, it leads to people doing stupid non camel cased class names - ie "Foobar".

It's a pity Text_Inflector never made it into PEAR :(

I can't decide if the restriction should be relaxed - just trust the user input to be right, and raise exceptions if a class doesn't exist - or enforced.
it does, but it overrides an API level 11 method:

http://developer.android.com/reference/android/app/Activity.html#invalidateOptionsMenu%28%29

i just followed lint's output. without these annotation any project embedding ABS will fail linting, even though in practice it will work on 7+. that's what the `@TargetApi` annotations are for (telling lint that it's actually ok and temporarily changing the sdk level)
can we remove this console.log?
maybe mentioning that no special params are required for the registered services
Need to return this. Also same comment with the error handling as with group.dataload.js
@normann I don't really follow your logic. You don't have to touch the conditional checks in the parent implementation when overloading getStatusFlags() as you suggested, so its "untouched" either way. If he wants to remove a flag added through conditional logic in the parent implementation, that's fine as well. Just as any modifications via updateStatusFlags(). No reason split out getStatusClass(), its over architecting, and harmful because its putting more view code than necessary on the model layer, please remove it.
Oops. Updated branch.
indeed :) done.
This will be done at the integration check in.
I'm pretty sure there is some sorting applied to the blog posts, but again, I'll have to look and make sure. If there is not, that is definitely a bug that I can fix upstream.
OK, but we should test for classes in SonataDoctrinePhpcrAdminBundle not for classes in SonataAdminBundle. I don't know what would be Symfony "best practice" or a solution on the dependency, but this code still has the chance to create another WTF experience when the classes are there by accident (e.g. installed by composer) but the bundle is not activated in AppKernel.
Here we should call deleteWaypoint instead of implementing the deletion directly.
This is already done below, forgot to remove this comment.
This function can become 2 lines with no conditional

```javascript
$('#inserterbar_action_add_page_container', $rootel).toggle(!isWorld);
$('#inserterbar_action_add_area_page_container', $rootel).toggle(isWorld);
```
Hello, Gary.
Nevermind about **they will be sent immediately**: we'll push it into SmartLifecycle.
>You can't "hide" the TS

But you are doiing that: ```TcpInboundGateway#getScheduler()``` :-)
Or I missed something again?
Not needed at all; already asserted in the outer class constructor
Right, but if __none__ of the set of supportedMethods can have a payload, and they specify a RPT, shouldn't that be fatal?
Need to assert that idCache was cleaned up ??
Looks like STS config issue - @ Override on interface methods is JDK 6.
These lines are too long which will look differently on different platform. Especially in VI/VIM, it will be hard to scroll down. The right way is to add a \n at character 80.
yes :) 

my little experimnet.. #fail.
Maybe use `_.defaults` to make this more concise? Maybe filling in defaults should be middleware. Also, I wonder if we aren't taking advantage of the middleware architecture of restify. I think you can pass an array of handler functions to a route handler and use the `next` callback in each function to execute them in sequence. See the two sections here: http://mcavage.github.com/node-restify/#Common-handlers:-server.use().
please removed commented code -- no need for it.
No need for JavaScript in HTML. You can move this to external file.
Why do we need the double-slash here?
So we're consistent, can you remove this extra newline to be consistent with the block below ended on line 69.
will these leave trailing whitespace between `arch` and `-os`? Also, its probably just me nitpicking, but the pe-mock-config erb trims all whitespace after variables, but not mock-config.erb. Perhaps using the same format across both would be cleaner.
@apex13 That issue should now be addressed.
nit: `getApplicationsForUA`
The issue here is that you have lost some functionalities by changing the api quite drastically (which also make it *not* backward compatible). Why did you rename the method in the first place? I actually don't like ``prettyPrinted`` as it is past tense and does not match any of the other apis (for example, the other one should have been called ``compactedRepresentation`` (for following this new trend) => it is not consistent)

Here I would like to:

* use the old name (``prettyPrint``) since it was fine in the first place and does not require any code to change
* keep the other api (the ``prettyPrint`` with indent value) and implement it if possible. I hope it is... If not possible then throw a NotSupportedException
* I also don't like the name ``compactRepresentation`` as it is not an action (methods are action). An action would be ``toCompactRepresentation`` and is consistent with other apis like ``toExternalRepresentation``
Sorry, I meant to say split on comma _or_ whitespace.  That would cover it, right?
please use an early return instead.
Using Kernel#system should be easier/safer/better http://www.ruby-doc.org/core-1.9.3/Kernel.html#method-i-system
Why this code here? What cases are we preventing, where the request host is different than the application host?
Not exactly sure :) I guess @pmuir knows.
oh lol the method actually returns the same thing, was integrating Ayoola's saving method and was using his variable naming, shoud I use the line 

this_questionnaire= get_questionnnaire_obj(questionnaire_id)

OR just 

this_questionnaire = get_object_or_404(Questionnaire,pk=questionnaire_id) ?
Why not pass in a question group rather than an id? That way you don't need to get the object from the db
Better yet you could rename this field to question_group, and then test to see if it is a questiongroup eg. isinstance(question_group, QuestionGroup)
and if it isn't you can then allow users to put a question_group_id which you could then lookup as a backup
Counter-proposal, as there is a role that fits the purpose

```
<section role="dialog">
  <div role="status">
    <p>The alarm is set for ... </p>
  </div>
</section>

```
"twitter": "basvasilich"
The 4155 test case had a problem with it.  As I said, I'm not 100% sure this is correct but from what I could tell this if block was chosen while the the expression was being evaluated as normal, and needed the extra jpe to handle the nan case.  It seemed as if the comsub code does not apply to floating point values, at least not in this case.

Moving this condition back to here and running the test cases should show how this fails, I'll re-reduce it for you later but I'm not on my dev machine at the moment.
Should read `were added by the tiapp.xml`
Vestigial problem from a different attempt at the fix, will remove
This is subtly wrong. `deleted` is part of the encrypted body (`payload.payload`), not the envelope. Shitty naming bites again.

Try this:

```
    final Object del = p.get("deleted");
    if (del instanceof Boolean) {
      this.deleted = (Boolean) del;
    }
```
Keep (system, package, apps, module) import order please
remove this dependancy, bring the 2 used classes into this codebase.
agree, and stringToInt as well?
xui.js has been removed
For XHR, we do not need to ACCEPT or DENY, we let the whitelisting mechanism take over. We only need to stop navigation to non-whitelisting sites as the browser won't do this on its own (Unless we want to changes this as its somewhat bizarre from a browser perspective).
Discussed with jeffheifetz.  Leaving this in for now, but opening tech debt blackberry/BB10-WebWorks-Framework#135
@work we use underscores for all private variables within a class so it's a habbit of mine  - should i now define _instance (available over a getter) and logger (not available over a property) or just skip the _ everywhere?