labofsahil/github-event-dataset-2012 huggingface dataset:
This method duplicates randomString().
According to https://developer.mozilla.org/en-US/docs/JavaScript/Reference/Global_Objects/Array/sort it shouldn't
The title is a string already.
The instructions to build and run are in the comments of the pom.xml - what more could you want?
Yeah, good point.
You can't use static ids because there can be multiple entries on a page.
OptString with an Int default.  Should probably be an OptEnum with "32" and "64" options.
You're only reporting one port, right?

If you're not to report all the ports found in the configuration file, this loop isn't needed, which would do the code more readable.
oiasldkas
yeah good catch -- not sure what i was thinkin
externalesh kon, zepto ro ham ezafe kon
Personal preference I suppose, but I think making this and below just `-1.0` makes this example cleaner.
it would be nice, since u added this extension, to have a test that evaluates it.  I mean...for the sake of adding tests to every functionality or extension we add, this will be awesome. What do you think?
Do we really need the method below?
Cap.
Mobile Web supports this property
The docs don't mention anything about those properties needing to be set before show is called. You either need to change the docs, or change the code.
Line too long, original was beter?
should be ````generate:command````
We do not need to use the DS constant in these sections: http://alanhogan.com/tips/php/directory-separator-not-necessary

Also can you remove the spaces around the periods? 
```php 
$markdown_path = path("storage").'/docs/'.$markdown_file. ".md";	
```
Ahh, no I clearly didn't - in my defence I did at least leave the untested flag in place on that one!
K, I might have a better solution:

```scala
val trimmed = Option(v).map(_.trim)
trimmed.filter(_.length > 0)
````
Fixed in latest commit. I use `to!RetType` for string types, and `array` otherwise.
You should use BeanManager.resolve() here.
This is an inefficient way to do this, sadly. You're using progressively *more* memory on each batch.

`subList` does not do what you think it does. It retains a reference to the original `List`, and a little more memory besides. And you're chaining them.

http://teddziuba.com/2008/09/java-sublist-gotcha.html

Give up on all this: just call `bulkInsertNonFolders(nonFoldersToWrite); nonFoldersToWrite = new …`.
Unfortunately no, due to the possible type conversion (see new commit)
The problem here is that GrapheneWebDriverFactory does not set any contract in its createInstance method which would set that created driver is proxy.

I would rather check if proxy is instance of proxy - if it isn't, proxy will be created - otherwise existing proxy will be used.
I don't know what I was thinking about. ;-) I realized it is right approach being more explicit. :-)
For MTL4 you need to download the sources manually because they're no longer available for old versions.

This is trickier, because you also need to rename the file.

What we need is a way to tell to EasyBuild what kind of file this is, because it can't rely on the file extension...
In my opinion I don't think that is simpler. Also, it is useful to be able to tell whether a redirect contains regular expressions via a meta query.
`pure`? I'm only asking because I see you marked other functions as such.
We do rely on the <i>transactional</i> sub element of the <i>poller</i> element for inbound channel adapter and we proxy the <i>JpaOutboundGateway</i> class if we have the <i>transactional</i> sub element to the outbound channel adapter/gateway.

The <i>@Transactional</i> annotations are no longer there in the <i>DefaultJpaOperations</i> class.
Do you know all databases? The WHOLE thing? :P
@superdweebie it's because you've opened the PR from master... Please don't do that next time...
Gustaf is off an on working on a Gerrit SCM extension that will require a jgit dependency, so thats no problem for us.
But beware, there is a dependency to the Git Plugin that has a dependency to jgit. So there already is an indirect dependency to it, and I don't know what effects it will have when a new version dependency is introduced in the git plugin.
So walk carefully ;)
Maybe we should rephrase this to "Push the changes you made to..."
This line is good...
That's the one! Tnx.
Not good grammatically. As a compromise, I suggest, "The basic application structure follows:"
Is it a no-no to duplicate these under the other headings?
Nope, leftover from the work I did in there. Removed it.
correct, those 2 methods are not use and are specific to DoctrineORM
Removed the unneeded declarations alltogether, and added resetting instance variables into tearDown()
wiki:
- remember that conditions with the same value ElseGroup will be used to make logical AND check, so to allow different targets for the same spell effect you have to set ElseGroup respectively.
But Shauren say me in irc: GetOriginalCaster is not available in Load :-/
nvm, I loked at it too fast
Mnja.
Use proper spacing between commas.  msg should be "Location should only be included if there is a 'url' field on the serializer"
Is this needed?  This data is already on this offer object.
PR [#9](https://github.com/armstrong/armstrong.dev/pull/9)
remove or update the comment, this is talking about the connection extension
baraye isole shodam kar `div.slideshow` ro zekr kon
ok, but it looks like we don't need this line anymore (after all it is commented out) so lets delete it.
I don't see how this code relates to the above commit, other than that both implement an `options` method - the actual code seems to be completely different?

Other than that my only comment would be that I'd prefer to see monkey patch somewhere more generic - either as an initializer or maybe in the test helper, rather than in a specific test like this.
`![]()` is only used for embedding images.  linking to external pages does not need the `!`
Sure. I'll do that at the end with the finishing touches.

This field is not supposed to be edited manually. We need that "hidden fields" feature in the streams_core. 

An alternative would be to create a simple parent_id field type with a drop down form to allow users to switch the parent on a specific entry.
Modern browsers don't show these quotes, but IE9 does. They appear to be spurious.
The KitchenSink repo currently has to support both 2.1.3 and 3.0, and in 2.1.3, "dragend" doesn't exist. That's why the following conditional is in there. I just changed the event name (to make the sample work properly) and the variable name (for clarity).
It depends if you need to enforce https on all communications or not. Also, depends how secure you need to be. Rack-SSL only does a redirect to https, but it still accepts the request.