labofsahil/github-event-dataset-2012 huggingface dataset:
yeah, the normal bash error is pretty good:

    test/run.sh: line 12: dart: command not found

that's one of the reasons I switch html5lib to this. The other way would give you an error about /bin/dart if you didn't have DART_SDK defined which was a bit confusing. Basically I think right now, users need to add the SDK bin directory to their path (to get "dart" and "pub") but they also need to define DART_SDK, which is two steps. I think we can get rid of DART_SDK: http://dartbug.com/5188, and only rely on setting up the "bin" directory
Another idea is to use the convention to always use a prefix:
- watchers.dispatch()
or 
- watchers.notify();

then we change: watch(f, callback) into 'add(f, callback)':
- removeFun = watchers.add(f, callback);
```
  this(true);
```
maybe remove this annotation instead of only commenting it
oh, in that case keep the list!
`ofRectScaleMode` is a shortcut enum used with `scaleTo()` (that has been discussed previously) that simplifies and combines scaling and alignment.  Each represents a combination of `ofAspectRatioMode` and `ofAlignVert` and `ofAlignHorz`.
(->> ::file .lastModified)
Let's surround this with {   }. Even though this works, it will make for a cleaner diff in the end result, and less coding if we ever have more that we want to do in the else.

Optionally, we could simply return false at the very end of this whole function as a catch all instead of having duplicate elses.
Não sei se o scaffold disso vai funcionar.
Se não funcionar é melhor escolher apenas uma entidade para ser o belongsTo e a outra vira um campo. Mas testemos antes...
Looks like this will fail, as "core" will not be set if -c value is not provided.

eg. ruby-1.9.3-p0 :001 > "#{foobar}"
NameError: undefined local variable or method `foobar' for main:Object
I don't think you should set explicitly this values as this means that any changes done in the default values of the bundle would be ignored becasue you force the config. In many cases, you don't need any config for the bundle
it would not :)

```
<?php

var_dump(!empty($config['request_content_param_converter']['enabled']));
```
What is the purpose of these properties?
thanks gcheng. Created https://github.com/WindowsAzure/azure-sdk-for-net/issues/20
Looks good and merged. 

Thanks! 

:::::::::::::::::::::::::::::::::::::::::::::::::::::::::

Eric Barnes
ericlbarnes.com | @ericlbarnes



On Oct 15, 2012, at 8:49 PM, Jesse Schutt <notifications@github.com> wrote:

> In readme.md:
> 
> > @@ -42,7 +42,9 @@ The `{{ email_form }}` tag accepts the following paramaters:
> >  * **to**: The form recipient's email address.
> >  * **cc**: A cc email address.
> >  * **bcc**: A bcc email address.
> > -* **required**: A pipe seperated list of required fields. Example: "name|address|city". Currently this only does simple validation to check if it is an empty value.
> Just corrected spelling on "separated"
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
A mi parecer estás sobre diseñando, el parser ya se debería encargar de esto, Lo que podés si hacer es un metodo estatico en asiendo que te devuelva un nuevo asiento con los parametros que le pases
evtl noch:
if($comment == ""){
  // Error: Keine Begründung angegeben
}
maybe we should move screenshots inside of the repo?
Remove the space in front of  `: `, capitalize Linux.
Does the performance actually "degrade" in this case, or is it just slower all the time (but consistently)?
this message looks wrong for people for which the recommendation is fulfilled (imagine the message being flagged as ``OK``)
See [here](https://github.com/wlanslovenija/PiplMesh/blob/master/piplmesh/frontend/templates/home.html) how you solve this.
Not liking this.
Do you want to change this to the code tag? 
<code>sudo easy_install pip</code> with <code>sudo pip install virtualenv</code>
No need to use a class variable here.. You probably just want an instance variable `@movies`
I think the second looks better.  so the saveNewCustomer is gonna be a method from the customer class right?
This we'll need to resurrect by asking Andrew where it lives.
Spaces before the bullet points do not render nicely in the HTML output.
I'd appreciate it if you could describe what this tag accepts in the class DocBlock or the constructor's DocBlock.
Thus what the format of the tag is and what version formats it supports (this is duplicate to the contents of the RFC but a link is troublesome at current because there is no formal location)

Another option would be to @see to the documentation page: http://phpdoc.org/docs/latest/for-users/phpdoc/tags/version.html
(yes, it still needs to be written ;))
thanks stof, one never stops learning :-)
interesting feature, i guess i have to update a couple of places
# Test

test note to challenge GFM.
@merk : AFAIK, validation happens when working with forms, not when writing data directly into an entity. The "encoded" version that's currently written into the database is definitely *not* a valid URL.
But you are right that I should have called the validation component instead of filter_var. I'll modify the pull request accordingly.
On 2012-06-18 07:07, David Precious wrote:
>> @@ -234,12 +245,9 @@ Specify an arrayref of columns that should show 
>> up in the list.  Defaults to all
>>
>>  Specify a template that will be applied to all output.  This 
>> template must have
>>  a "simple_crud" placeholder defined or you won't get any output.  
>> This template
>> -must be located in your "views" directory.
>
> Were the changes above intentional, or were they the result of
> working based on a CPAN version rather than the version in master in
> the GitHub repo?

Not intentional.  Definite CPANism.


-- 
      Michael Rasmussen
    http://www.jamhome.us/
Be Appropriate && Follow Your Curiosity
I've been burned before trying to keep things in a TestCase's self, so I've started doing this in reflex.
I can rework this with little trouble
But shouldn't this be handled by our library, not user code?
Hrm. You are probably right about the purged thing. Here's the use cases I definitely want to support:

  * Service configuration is present and service is running.
  * Service configuration is present and service is stopped.
  * Service configuration is absent (and service is stopped, of course).

Make sense? I don't think the old code supported that correctly, though, now that I look at it. So yeah, start with changing `default` to `present` and then adding a default block that fails if `ensure` is passed an invalid value, that should be enough to address my concerns in this block. :smiley:
I think your missing puppet-lint
It's simply that I do not use it anywhere else. If we keep it, then please add some assertions as to whether the fields have been set correctly.
The phpdoc is missing here
It seems that you are using an old version of XMLEditor. CAn you redo the patch with version 5.1.1. Both Gunnar and I have aligned to avoid this annoying space issue.
It's acceptable to call a collection of "foo", foos, so that could just be `rows` if it makes sense. Like I said, I don't really know what's going on up there.
why i this default?
I think the list is implicit as by default a menu has type="list".
no test?
If anything this should be ```label = interface.pop('label')```
New README content added explaining an example model/admin configuration for a polymorphic inline.
I think the page parameter name should be defined for the collection as you may have different names in different places of the app (especially if you use it in a shared bundle, where you cannot control the global config)
can you use array.indexOf ?
needs a rebase
Do we need to clear the group uuid GP here?
CS you should insert space beetwen if and ( : http://cs.sensiolabs.org/
Please reset the state of the cmd object between test cases.