labofsahil/github-event-dataset-2012 huggingface dataset:
Here you don't specify the sbt version the user has to use, so depending on which version of sbt do you have this code will potentially fail.
On my machine I have sbt 0.11.3-2, if I load this project, it will fail because it can't find sbt-package-dist_2.9.1_0.11.3 version 1.0.5

If you provide a script (just copy paste the one in finagle) that run a specific version of sbt (0.11.2), it will succeed because sbt-package-dist_2.9.1_0.11.2 version 1.0.5 is available.
Энэ орчуулгын хэллэгүүдийг ярьж тогтох хэлэлцүүлэг хаана хийвэл зүгээр вэ? github дээр форум маягийн юм байдаг уу? Бүлгэмийн мэйл листээрээ ярих юмуу?
But what if I WANT to check for inclusion of string "%nick%", but not really my nick?
I wasn't supposed to be! But yes, you're right, it's at best redundant.
Doesn't seem to send :clear through.
yeah, maybe this should be get 'people/webfinger' or something....
I confess, I'm not hip to the state of the art in busy-wait loops in ruby, but shouldn't we perhaps be sleeping a bit in the loop instead of repeatedly immediately asking "Are we there yet?"
A cause de la technique Boostrap de "wrapper" la checkbox dans son label, je n'ai pas eu d'autre choix que de faire comme ça malheureusement.
J'aurais voulu écrire le label avec la technique habituelle en Rails, mais ça fait n'importe quoi, et je n'ai trouvé aucune solution satisfaisante sur le net.
For future, let's get this alpha sorting func into our API since we now have two widgets doing the same. Not a blocker.
This isn't quite up to Sears standards. Please try again.
@NewpTone can you clean up this indentation?
Please, try to keep description line lenghts shorter than 100 columns.
Changing this in the library code is pretty dangerous. Will require some serious testing of other users of this method.
please give me the example to follow on
```payload``` is special in exploits, don't use it.
This is already available via Msf::Config.install_root
For volumes we need to validate that "mountpoint" and "size" is found in each element of the list (actually I'd love it if we tested all of the values using regex's at least but since this test was already there I won't hold up the pull request for it).
Can you also please change these examples to have just one update function that checks if `_sp` is defined or whatever instead of having the separate `_update`.  The division can be confusing.

I initially was also going to say to only create `_pickId` if it is a pick pass; however, since this is a lightweight object and a simple example, I think we are OK without the check.
@shunter I think I asked before, but is `Object.freeze` fair game now?  We don't care about the performance implications?  Perhaps, these objects aren't used often enough?  Or browsers recently implemented it nicely?
Do we cache these allocations later?  Probably something we want to consider.
I'm not really sure.  Most people would just use the Image object directly.  I think the bigger question is should we always be setting the cross origin flag?  Or should the user have to explicitly state they want to allow cross-origin?  @shunter, thoughts?
Why have a max time?  Click the moue button, pan around a little for 2+ seconds, then release the button while moving.  The globe stops instantly rather than having inertia like it usually does.  That doesn't seem desirable to me.
We should really tell them about this when we have time.
`callbacks` here should be an Event, which encapsulates the same logic.
If we need a getter and setter for a property, we usually set it as property accessor for the proxy and that will generate getter and setter automatically. Something like https://github.com/pingwang2011/titanium_mobile/blob/master/android/titanium/src/org/appcelerator/titanium/proxy/TiViewProxy.java#L36 .
I didn't know you could do this in python. That is just insanely handy.
use 

```python
"\n#### %(letter)s (%(count)d packages)" ...
```
That is uncomfortably long; could we have an initial wait of ~ 1 second instead?
what about deleting all the elements in `this._pendingQueryCallbacks`?
Ah, sorry. I don't think Connection has a concept of 'connected' but connected is set to false in the ConnectionInPool constructor (which extends Connection).

'self' in this line refers to an instance of ConnectionInPool so I think this should be fine. I could be wrong though, not entirely sure of the division of responsibilities between ConnectionInPool and Connection.
Should this be:

```while( offset <= length - 2 ) {```

I'm trying to parse incoming multipart messages, and if you get a segment with a single line of header then that header gets skipped.
I know it's not 100% accurate, but I think streaming device was a pretty bad term too... I'll rename it to `isFileHandle` though.
Why are you calling `toUTC`? That has _zero_ effect on `stdTime`. `toUTC` returns a `SysTime` with the same `stdTime` but with `UTC` as its `timezone` rather than whatever it had before. _No_ conversion is taking place at all. The call to `toUTC` is pointless and wasteful. This should just be

    return stdTimeToFILETIME(sysTime.stdTime);
We'd need to make this (provided we have forward iteration):
```D
_current = _items.front.save;
```
To be able to make joiner a true forward range.
Relevant: https://www.google.com/search?q=cannot%20vs%20%22can%20not%22 According to a few of these entries, "can not" is a lot less used than "cannot".
Instead use `assertThrown(singleSpec(""));`.
That sounds much better ;) I made the assumption that having the constant in the source code would invoke it and fail.
there should be a space i.e. ``) {``
Attention, il n'y a plus de `factory` dans atoum, donc il faudrait mettre à jour le code s'y rapportant dans la documentation.
Don't do a default _head.tpl file.  Let `PSU\Template` do all the work for you.
I am generally open to this but atm its not like this bundle has seen a lot of love lately.
Commented method - keep or delete ?
Lines 222-225; the <Summary/> is actually 2 sentences, Albert is saying that the guideline is just 1 sentence. Honestly I don't care on this one, and prefer longer if it's worthwhile information..
You don't need the name or the url methods either.
This is kinda non-sense copy-pasta error
I think its better since it avoids a pointer cast (which gcc can get picky about).
Theoretically, this should already be covered by the replacement two lines above:
``JMS\SerializerBundle`` -> ``JMS\Serializer``
-1 :hand: this makes the code less readable.
In that case, can we leave it as a div and give it a tabindex so that you can tab to it?
Instead of running this in a conditional, you could set a variable to the result of this, and then just use `.toggle()` which takes a boolean to show or hide -- when true, it shows, otherwise it hides.
@croby Should be fixed in a follow up commit.
These should be `{}`, not `(}`
You made the separator darker. IIRC, gray is equal to #888, but the color was originally #ccc.
No need for array merge. I'd instead directly store the values in here :)
Looks like spaces. You need to indent with tabs.
Fine by me, will refresh the pull request.
I think the assertions should be case sensitive, and we should remove this particular assertion (`assertFileNotExists(destDir, "class_caseSensitive.html");`), so long as we're testing that the `object_caseSensitive.html` file exists it shouldn't be a problem.
Ah, after looking ahead, you need to add it to every function definition below.
wanted it to be so there was plenty of room on both side to fit in other things :)
@christianv Can we remove the link classes on these since they're no longer clickable?
Yes I need to hide the block first without the slide animation
most of this method, except for this line, is identical to the one after it.  It would be nice for the actual tests to be something like this:

```
@Test
public void evaluate_shouldReturnTheTheFirstDateAPatientWasConfirmedHIVPositive() throws Exception {
	Obs obs = new Obs();
	obs.setConcept(conceptService.getConceptByName(MohEvaluableNameConstants.HIV_RAPID_TEST_QUALITATIVE));
	obs.setValueCoded(conceptService.getConceptByName(MohEvaluableNameConstants.POSITIVE));
	obs.setObsDatetime(this.generateDate(2012,Calendar.DECEMBER, 10, 0, 0, 0));
	
	this.createEncounterForPatientWithObs(PATIENT_ID, obs);
	
	MohConfirmedHivPositiveDateRule mohConfirmedHivPositiveDateRule = new MohConfirmedHivPositiveDateRule();
	Result result = mohConfirmedHivPositiveDateRule.evaluate(null, PATIENT_ID, null);
	
	Result expectedResult = new Result("10/12/2012");
	Assert.assertEquals("Obs date comparison was not successful", expectedResult.toString(), result.toString());
}
```

... and then you can write the `generateDate()` and `createEncounterForPatientWithObs()` methods separately for reuse.
m_coordinates and m_document can be readonly.  m_id probably can be, too, if you get rid of the setter above.
Use `List<Cartesian>` instead of `IList<Cartesian>` on the left-hand side, too, for a tiny performance boost.
Add a new parameter to not update the entity (because the attachmentdata will be wrapped in the widgetdescriptor)
`add` and `remove` sound good, yes!

Just to make things clear: I don't plan on doing any work on this, so it's your job now to draw from some other ideas! :)
Did you want to move the helpers to the root directory for others? Maybe `/helpers/`?
JSlint не знает в какой среде используется JS поэтому ругается - можно ему намекнуть - `/*jslint browser: true, devel: true */`
sorry if this has already been debated, but what’s the rationale behind these class names?
`i-edit`, `i-add`, `i-arrow-left`, `hdr-clear-search`…?

As you know, we’d prefer to avoid arbitrary class names as much as possible.
Removed the hashcode, it was autogenerated code from netbeans and unused anyways
zepto markazi plz
You can just declare this on first use
> Cool, but the rest of the project has 4 space tabs.
Fixed, also: It should actually compile now :)
@wireframe I use the same editor but different command line flags for my git editor.  I'm cool with holding off on that change due to complexity.
Please fix the class name. ``/`` is not the namespace separator in PHP.
Raised tickethttps://jira.appcelerator.org/browse/TIDOC-418
I've made those changes.
ich würde das nicht moven sondern symlinken. falls das erzeugte binary auch als command verwendet werden kann würde ich es sogar nach bin/irgendwas linken und von dort aus verwenden. wenn es aber sowieso nur als resource verwendet wird ist der pfad ok.
There is a bug here, as a queue name can be overlapped. So we might need to return to the original proposition, or find another solution. I don't really like the first one because we add semantic to solved a key name.

maybe the definition should accept a ```default`` option

```- { queue: transcoder, routing_key: start.transcode.video, default: true }```

and then, when we can have 

```
foreach ($queues as $pos => $queue) {
    $amqBackends[$pos] = new Reference($id);
    // check if queue is the default ...
}
```
This will break the functional tests for invoke and invoked
not off topic @jeffheifetz yeah it used to do it but I think one of the rules I made it ignore also made it ignore this case, we should open an issue for it and patch/update cpplint to follow this rule
do we need this still?
Yea, thanks for the tip, I obviously messed something. I am fixing the PR now.
It's a parameter and it is called msg
You're right, but that code shouldn't have even made it in, so it's removed in 51f444d9
You can still update the blog post :)