labofsahil/github-event-dataset-2012 huggingface dataset:
Could we update all files to the new company name and then check for "Red Hat, Inc."?
n_times please ;)
man yadame ke to design hamishe migoftin vase css ha ye foolder be name stylesheet baz konid va copy konin onja.
alan aksare site ha ro ke didam foolder daran baraye js va css
manam ro hamin menval in kar o kardam ama az in be bad chashm be doone foolderesh mikonim :D
shoma joon bekhah :p
@uwej711 the idea is good but i think at the time when this code is running, we are building the DI container, and can't rely on the sonata service already being available. on the other hand, if somebody really needs that bundle in the namespace but does not use it, he just has to set use_sonata_admin to false.
Setting $match should be doable using functions instead of loops. But it's not -- we need better list and string functions. >_<
What about adding an exception in here to catch invalid input before we wander off and do something strange with no debugging? Since escaping is hard it seems likely someone could put the wrong type of slashes in here or the wrong number. Could we catch if length is zero, nil, or greater than 3 and say something useful for those people here?
No need to assign this to `itemsAdded` -- `$.merge` does that itself

http://jsfiddle.net/6Seym/
This should be spree/users/affiliate.
Maybe, you are intended to avoid dynamic casting (== vtbl lookup overhead)?
if it is true, this is reasonable.
I mostly did it this way for consistency. In other parts of `core.thread` we use the approach where we define a simple `op` function.
ok; but I don't see why it hurts to leave it protected. Your call.
I think where I'm actually getting thrown off here is in the first *else* of the method. I'm going to add a question there.
Why did you need to add this else clause?
are the category, region, sort by part of the search? if so, should the button be at the bottom rather than above them?
I know you probably copied these lines from my code (maybe?) but if we can keep consistency to put space before the asterisk that would be awesome. NSString *myString... (instead of NSString* myString)

If we can do this from now on that would be great.
`#` denotes instance methods; `::` denotes class methods.  This comment applies to your other spec files too where you test class methods.
@apex13 Can we avoid JavaScript confirm boxes? Is it possible to find another solution for this? (e.g. pop-up)
Не, гайд этот только к первой. Во второй вот такой CS:

https://gist.github.com/2835928
https://gist.github.com/2835927
имеет-иметь в одном предложении
better is "".equals(picUrl)
*doesn't* -> *does not*
fixed both...
Handling the close event fired when transport is removed. Without this my tests hang due to the open socket, also kinda nice to free resources.
Isn't this really bad? By altering the order of the list of values in the enum, aren't you effectively reassigning the meaning of existing seq_types in the assembly table?
One option is to not match against each specific block but simply do this:

    Enum.each blocks, fn(verb, code) ->
      quote hygiene: false do
        def handle(unquote(verb), unquote(path), _query) do
          unquote(code)
        end
      end
    end

Then later if we want to support other verbs, we don't need to change anything here.
so,  +1 for filtering first
to avoid this kind of "ask don't tell"

use (in the Translation model)
===>

delegate :yaml_path, :to => :phrase
Can we move the `@user and @user.moderator? and params[:show_redactions] == "true"` into a shared `show_redactions` method or something given that we using that test multiple times.

Ideally we want to be able to share it between all three controllers I guess, but moving it into `application_controller` doesn't massively appeal to me I have to say... Not that what the best answer is.
I don't think so; the dropdown doesn't activate on other pages - otherwise it would give you editor links on other pages too.
#resolved
(minus the part of the sentence I accidentally left at the end)
> is an
some comment why this method is skipped would be nice
My mistake. Milliseconds. Ignore above comment
These should be `startsWith()`.
Since the real trace() call can't be used in the way I was describing in my explanation.
You need to free the result:

```php
$db->sql_freeresult($rs);
```
WTF ? Admin classes are created using the DIC. Why would you do the dependency injection by hand for this one
the description in the it explains this, no need to write it again
unnecessary comment
A BOM should have no parent. 

See arq-bom for example: https://github.com/arquillian/arquillian-core/blob/master/bom/pom.xml

(it also means it has to duplicate a few of the distribution type elements in the pom, but that's ok in this case. )
Just a quick questions. Currently you are adding CSS classes in both the TouchMoveHandler and the TouchStart to show highlighting etc. When the menu item is in a shown mode, are we removing this TouchMoveHandler? Does it default to one or the other? ie: Are we not setting them twice?

Would it be better when in shown mode to just use CSS? So we remove the Touch move handler? Correct me if I'm on the wrong path.
Code convention check in whole document
You should only use one comparison of the form:
~~~clojure
(<= some stuff here)
~~~
instead of ḿultiple comparisons
Large amounts of data are OK only if:

1. `select count(*) from table` works fast. In case of InnoDB and not using keys it can be really slow.
2. Page size value isn't too big.