labofsahil/github-event-dataset-2012 huggingface dataset:
classmethod was to match existing api, could go away with more api change. It probably only really needs to be the static method
i got confused .. never mind.
It throws error, I tried it before, charset has to be given otherwise uniConv throws NS_ERROR_ILLEGAL_INPUT. This was the reason for having an encoding detector in the first place,
```
[jens@ca60c173 espresso-3.1.1]$ make test
make: *** No rule to make target `test'.  Stop.
```
So just remove this comment then.
drop `"bin"`
If you agree with splitting the new refactoring and the source generators (https://github.com/scala-ide/docs/pull/19#issuecomment-6600933), we probably want two different "feature boxes", one for the ``New Refactorings`` and one for the ``Source Generators``. Also, the ``Move Template`` box should be merged into the ``New Refactorings``.

I can do it myself, if that's ok with you.
No, fixed.
this is actually ``instruction``, not ``action``, right?
> It's not about compiler semantics, It's bad coding style because you have to **remember** the rule.

Looks like your personal opinion as I have never heard this before.
But lets think about it. If behavior here is undefined, the compiler will obviously reject such code just like it rejects `int a = b, b = 2;`. This rule is same even in C++. And for the reader this code is having a single meaning so, if the reader didn't know about this rule, such code will just help him to learn it and be happy. So I see no reasons against this code.
This is the runtime for D2. The Tango port for D2 works alongside Phobos. I don't see why you want to special case anything for Tango here.
MINOR: comment refers to file as "all_tests.h", but actually called "all_tests.cpp"
I'm not quite following what type of situation would end up in this final else. Can you explain?
Ok, I'addressed all comments by either commenting or fixing. You can see it here https://github.com/olegz/spring-integration/commit/10b629af9e57e536a236819a90ef22fbefb2c0f6
That's why I used uri.toLowerCase().startsWith()
Recommended cache life is 60 minutes or more because the server's data is cached 3-4 hours.
that's wrong, antiscroll requires `overflow: scroll`  ( https://github.com/LearnBoost/antiscroll )
you should remove the extra space before ``::``
Damnit, let's not do the one line if statements. they're really hard to read.
OK, changed. Thanks for the feedback.
Minor point: Controller::join_links() should be used to avoid double slashes (edit//2).
This comment won't document anything.
Will this code block ever get executed?  If (match > 0) we wouldn't have ended up in this else-if block.
Please do a typeof check and then use toLowerCase. While #224 hasn't been resolved yet, most params are case insensitive.
mmm, corregi eso q esta mal
All this WHITE_BOX stuff should be retired. We only put it in tactical so we could get the tests working with the move to RDL.
I would prefer an integration test with the plugin controller instead. I'm not using such tests anywhere else.
Does your version pass pay checker and linting? A lot of the clean up I did was to make the image writer more like the others and to cleanup errors. I don't really care if the writer is function or an object as long as the object makes sense and passes the tests.
Regarding method names, I think there is no need for an 's' in _requiresParametersValidation_. _requiresParameterValidation_ seems better imo
Perhaps `ApiClient` would be easier to understand here as a constructor.
Hmm... i think you reset too far back or something. This is going to bork the site if we let it go through.
Why escaping it as HTML when the output is a JS file ?
Certainly possible, but a significant amount of additional effort. You'd have to describe the object being queried/searched, consuming a network request and delaying the query when that describe might already be cached, and then compare against the fields on that object. Plus there's the case where you're chaining fields through a lookup relationship, so you might have to describe multiple other objects too...definitely a messy slippery slope. Does the SDK really want to be in the business of making network requests on your behalf?
Там  трабл з передачею кукі який  ми так і не змогли рішити.
i'd like to keep the cli functions pretty tight and this looks like a good place to extract a function.

```ruby
# github.rb
# @returns [String] socialcast username to assign the review to
# @returns [nil] when no buddy system configured or user not found
def socialcast_review_buddy(current_user)
  # TODO
end

# cli.rb
review_mention = if buddy = review_buddy(current_user)
  "Assigned to @#{buddy}"
end
```
Ok, but thats changing the original implementation.  It was outside the if above.
We traditionally say `PEBKAC` :)
South still works because at the top there is the import of all the models. 
The reason for fb.py is namespace. This lets you use fb.User which I think is cool.
Tämähän on nyt sama kuin:
~~~clojure
(apply my-map rest sequences)
~~~
Ei hätää, sinulla on kasassa jo 121 pistettä.

Ajatus on aika lähellä, mutta ongelmana on se, että `my-map-one-element` ei toimi oikein. Esim:
~~~clojure
(map count [[1] [1 2] [1 2 3]])
;=> [1 2 3]
(my-map-one-element count [[1] [1 2] [1 2 3]])
;=> UnsupportedOperationException
~~~

Lisäksi käyttämäsi matriisin kasaaminen käy paljon yksinkertaisemminkin. Pohdi mitä nämä tekevät:
~~~clojure
(my-map first (cons a-seq more))
(my-map rest (cons a-seq more))
~~~
and that 'it' has a value
@Priyansh This is testing that session.keepSource was changed to false right? But what if it starts as false anyways, then we aren't actually testing that our function changes it.

We should do a session.keepSource = true; before we parse. That way when we check that keepSource is false, it will be because of our logic.
Got included with the rebase.  Removed.
Daniel Craig is deceptive
Is there a way to put less logic in the JSPs? or is it just the way things are done when using them.
You shouldn't need a SSN to open a bank account at BoA, Chase, or Wells Fargo either,
But that stuff should be mentioned.
Why does it need to be updated?
I think this comment needs to be a little bit more clarified. Reading the code tells me it puts all modules in the safety state, but naming the function "update" makes me think that the safety state is changed (updated) which is obviously wrong.
arrayList?