labofsahil/github-event-dataset-2012 huggingface dataset:
Two reasons: I thought `getChar` only worked for single chars ;-) and because potentially we'll eventually allow non-Latin characters as hotkeys.
DRY this segment out
why? 'rest' is the default/current behavior
Well looks like this pattern is repeated into the code, then I'd suggest method like this:

    public function _parseOneEntry(&$container, $item, $className)
    {
        $instance = new $className();
        $instance ->parseXml($item->asXML());
        $container[] = $instance;
    }

Then you can call by:
$this->_parseOneEntry($category, $categoryXmlInstance, 'Category');

and for Person you can do that by
$this->_parseOneEntry($author, $authorXmlInstance, 'Person');
Make sure that all check style goes well. The if open bracket `{` must be on same condition line
This is useless as the DI extension will always replace this parameter
using `"bin"` is useless, it's already being checked in files
Oh of course. LGTM
I don't think this works unless the module has been pluginsynced. Check out https://github.com/puppetlabs/puppetlabs-corosync/blob/master/lib/puppet/provider/cs_commit/crm.rb for how to do this safely
yeah .. i think in that case lets override the parent constructor.

btw for the prefix thing i opened a ticket https://github.com/symfony-cmf/Routing/issues/38
Since you're on it, I'd suggest replacing this whole table with the actual country codes for each language (e.g. instead of strings, store Language::Id values there directly).  The way it's done here right now (hardcoded one-byte values for bad languages) doesn't look good.
can you switch `bind` to `on`?
We don't need it but imo it is best practice to have it so we avoid a possible collision with other code.
Apart from that, it also avoids the need to add `module`, `test` to

```javascript
/* global
```
Shouldn't this transaction decorator move to the ``import_data`` method of ``Importer``?
same whitespace comment as above, and again, the existing whitespace was acceptable (and desired) anyway
O case do marceneiro lida, entre outras coisas, com atenção a detalhes, esmero na profissão. 
Sugestão de leitura: http://www.objectos.com.br/blog/2011/06/19/o-case-do-marceneiro.html

Isto porque dei uma rápida lida no código do Stela e não encontrei em que parte a quantidadeMoeda e valorMoeda estão sendo utilizados.

Qual a diferença entre valorBoleto, quantidadeMoeda, valorMoeda? Para que servem cada um?

Outro detalhe é que na API que você está fazendo, o principal campo referente a valor financeiro (ValorBoleto) continua como String.
@aimonb in case of a compute node, what about enabling only 'metadata' as a default ?
not sure I like "site manifest site.pp". Ideas?
the function name is confusing, is it verifying store receipt or response data?
中括弧。全体的に中身が一文の場合は中括弧を省略してもらえますでしょうか。
参考 → http://docs.scala-lang.org/style/declarations.html
Also ternary operator chaining is a bad practice
change this to something meaningful -- can simply say "test file for MohGenderRule" but should not have all of this information, especially since it was not oliver that made it!
It's unusual to explicitly Close something that is also managed by a `using`.  Also, the `db` is closed before the `cmd`, which seems odd.  Is there some quirk of `OleDbConnection` that requires this?
@jpoyau, @wegrata 

While conceptually that makes sense, I'm not sure it will work much better. The db that comes into the handle function is too specific; ie http://<couch>:<port>/incoming, and it needs to be saved in http://<couch>:<port>/resource_data.  One would have to dig into the internals of the database handle to extract the db.resource, from which another Database instance could be created. See the [couchdb.client code](http://code.google.com/p/couchdb-python/source/browse/couchdb/client.py#273) for details.
Once #46 is finished, `#neighbor_params` should be able to look like this, i think...

```ruby
def neighbor_params
  allowable = allowable_params
  if user.access_level >= AccessLevel.super_volunteer
    allowable << :ssn
  end
  params.require(:neighbor).permit(allowable)
end
```
I don't know that this comment knows what he's talking about anymore
aaahh now i get it why the localname is relevant.

i pushed a fix to generate_fixtures.php now that should make the localname look right (it was totally random, not even properly eating out the namespace). speaking of namespaces: your code should handle nodes with the same local_name but different namespace fields. can you do the CASE on a CONCAT? i think the names in jackalope objects are including the namespace anyway, so right now reorder would fail with namespaced nodes.
Same as above, we can easily change this later. I'm more concerned about getting a working version out.
css haro alignment kon
ye pligin sade roo sl2
no, you should make the route extractor provide the needed data
yeah exactly that, the advantage is that it is quicker, if you already have the object, why do another database lookup when it costs you nothing to send the object to the function?!
can remove this first line and change primary_keyword_quantity by phrase_quantity
This part contains the real instantiation of the Spotters/disambiguators. The respective constructor is _only_ called if the spotter/dis. is supposed to be used because it is defined in the config.xml
files --> artifacts
@stof, I agree. Good idea.
Maybe specify that only location[zip] is required?
We are calling jake on the web platform to have it build the webplatform.js file. Would this be something we'd setup for them in advance, or should I move it out of the if statement?
We can shorten the signature for now, since we never use the full length one.
Is there a reason that you prefer calling bloginfo('template_url') over and over again rather than assigning the function result to a variable at the start of the script and using that variable instead? The latter seems more efficient since every call on the function causes it to be executed again and landed on the same value. What do you think?