labofsahil/github-event-dataset-2012 huggingface dataset:
Actually, this was the situation when I stopped writing unit tests and started refactoring the command in order to make it testable. Turned out that unit-testing symfony commands was quite difficult because they didn't use dependency injection but service location... (getting the container injected).
Therefore, I created the [console-extra-bundle](https://github.com/seiffert/console-extra-bundle), which allows commands to be defined as services. I would suggest we skip the command's tests in this PR and make it testable in the next one. What do you think?
@jzaefferer @scottgonzalez if you guys can review the above, just let me know please if you have any better idea to implement this.

After this change, it tests all the releases of the config file (stable and legacy). To accomplish that dynamically, I had to wrap each test function with a custom `this` scope, so the fn is able to access each release by using `this.release`.
I would rather use `razor.ip` here, since that follows the pattern for passing arguments to modules on the Linux command line.
Is it used anywhere else?
Yes, and it makes it compatible with the URL decoding Graphite uses -- which is Python/Django and we probably would not be able to change Graphite to become compatible with gdash. 

My intention was to be able to pass a regexp through gdash dsl to Graphite.

I am far from being an expert on Ruby but it seemed from the docs it should work on both Ruby versions. How does it break?
@Marlena  we are not doing a search here we are sorting all the extensions
define on line 87
are you suggesting adding the sed line to that block? or does it break during make as7? I didn't do much testing with anything as7 related unfortunately
Before anyone asks - yes, `$(LPAREN)` works in examples too.
`const(int)[3]` and `const(int[3])` should be merged to `const(int[3])`, because it is compatible with compiler's output.
This is in Levenshtein.distance, which is called by levenshteinDistance, which contains the restrictions that both s and t, are of type ForwardRange.

Side note: Shouldn't the struct Levenshtein be private?
don't stutter, use auto
This is an instruction to you :-)
Either document that `filename` is not null, or throw/check.
remove the comment area if not needed
do we really want to have those versions there?
I wonder if the REST API stuff shouldn't be auto-included as not every app will use it.  Perhaps put the example code in the wiki if it isn't already there and keep this startapp super light.
this will cause a warning on PHP 5.4 due to an array string conversion .. so we need something a bit more elaborate to deal with arrays for sure.
a typo - GetTypeAccessorMap
For all the examples for Storage and Service Bus, there is no mention of Service Management/Runtime expect in the intro. I'd expect at least a link to a How-To topic or something here.
The more I look at this bit (ClassicCheck) , the weirder it is. 

1) it is a class for no reason
1a) and it's instantiated every where it used

2) appends to the path every time it is used 
2a) and imports up2date_auth (and all it's deps) and loads the systemid

Going to see if I can do something more clever with the mocks.
Could you add that conversions to the compiler pass, and then set the converted data using the ``setListeners`` method?
Für die Leserlichkeit bitte folgendes verwenden:

if($order->getState() != Order:: STATE_CLOSED){
...
}

Gefahr des Verlesens bei == ist (zu) gross...
Yeah, I see the logic here.  OK, I used the assertNumQueries.  I ran it through the debugger to verify that it was doing the right thing and had the right count of queries.  Lots to count there!  I'm generally asserting the count of cases.  Does that seem like too much as well?
Problem comes when you have your editor to do tabs as spaces with your personal preference - be it 4 as in PEAR or 2 as in Symfony - and some other developer has it setup the other way. Then you either have to have an editor that will autodetect tab indention on a file-basis, or you'll have some funny looking commits. Tabs is the sane way to go about doing this.

As far as some editors not allowing you to setup what a tab looks like, find a better editor. Every major IDE/editor I've used has had this feature, so I don't think it is a valid excuse for not using tabs instead of spaces.

http://c2.com/cgi/wiki?TabsVersusSpaces

I would very much like to see where, in the context of indentation in PHP, how using tabs vs spaces can break something. I can understand that you might be writing Python/Ruby code where they uses spaces - 4 for Python and 2 for ruby - as indentation by convention, but this is PHP. Using spaces means I can't set my editor to use the monospaced number of spaces I want to represent an indentation-level as.
Looks like a copy-paste error, but your comments don't agree with the actual code.
The uglifyjs has `#!/usr/bin/env node` so if you ever run the script as a shell user (manually, deployment or a cronjob) it can be found through the environment. But I see now that my ProcessBuilder implementation will break if it's not set. 

My suggestion is that I default it to null and checks the value when creating the ProcessBuilder, so it's possible to build without supplying path to node.js if ran through a shell.
Is `puzzle_url` something that is left over from a previous iteration? Because I don't see it being used anywhere.
I am *fairly* certain, hasattr is bad style, and ``'read' in thing`` is preferred.
Sure Ill remove them, I had had them in there for my own reference 
before I made put the code up on github.

I have removed them then commited and synced my silverstripe-mobile branch.

Sorry about that.


Lee Blazek


  BERZERK* interactive


    Web, Mobile, and Mobile App Development

+64(0)21-55-63-55 | +64(0)3-443-4261

www.BERZERK.co.nz <http://www.berzerk.co.nz>

/*Attitude included no extra Charge/


    Our Latest apps

Queenstown Happy Hour - for iPhone <http://www.queenstownhappyhour.com>

Wanaka Happy Hour - for iPhone <http://www.wanakahappyhour.com>


On 5/19/12 10:34 AM, Sean Harvey wrote:
>> @@ -23,6 +23,13 @@ public static function is_iphone() {
>>   		return (preg_match('/(ipod|iphone)/i', $_SERVER['HTTP_USER_AGENT'])) ? true : false;
>>   	}
>>
>> +	// i added 14/1/2011
> Please don't add comments like this, version control is enough to know you've added these changes. git blame works well ;-)
>
> Could you please remove this?
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/silverstripe/silverstripe-mobile/pull/20/files#r848537
>
Everything in these 2 Java classes is a copy-paste from the ancestor class to make things work. The only useful bit is the redefinition of the function serialize(). Everything else is fluff :) taken as-is and strictly minimally adapted to make things work.

The usual way of providing a custom serializer does not work unfortunately when one is trying to provide a custom serializer for Map (it works and was designed for non "basic" types, such as BankAccount or Employee and such). Tatu took a look and confirmed, this was the way to go to customize serialization of descendants of Map unfortunately. I agree with you: weird :)

So this is a bit of a hack as is, but it will be taken out with Jackson 2.0
Same here... form? how should products be added?
this should remain as -SNAPSHOT until we release.
@merk Sorry about that, I was really tired yesterday. Thanks!
Jay,
I'll update the pom.xml before the review; yes we can 'hang out'.
Thanks,
Bruce

On Thu, Sep 27, 2012 at 11:59 AM, Jay Hodgson <notifications@github.com>wrote:

> In pom.xml:
>
> > @@ -15,7 +15,10 @@
> >  		<maven.compiler.source>1.6</maven.compiler.source>
> >  		<maven.compiler.target>1.6</maven.compiler.target>
> >  		<webappDirectory>${project.build.directory}/${project.build.finalName}</webappDirectory>
> > -		<synapse.version>2012-09-21-a6d6ad7-125</synapse.version>
> > +
> > +		<!--synapse.version>2012-09-21-a6d6ad7-125</synapse.version -->
> > +		<synapse.version>develop-SNAPSHOT</synapse.version>
>
> (synapse.version should not point to develop-SNAPSHOT)
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Sage-Bionetworks/SynapseWebClient/pull/95/files#r1710690>.
>
>
Peut être ajouter une phrase : " et le programme a donc échouer au moment du instanceOf". Je trouve que la transition vers le paragraphe suivant est un peu brutale :-)
Couldn't we generalize this as a is_loaded() function with an arguement for library?

is_loaded('mootools') or is_loaded('jquery') for example, if we use the filenames as determinants for what the files contain.
remove!remove!remove!remove!remove!!!!!remove!!!!!removeremove!!remove
This uses the data-button attribute in the markup, so that 'startButtonNode' can be the 'Start' or 'Search' button, depending. Otherwise, the "Start" button is always clicked, which doesn't make much sense if it's hidden.
Yes? I think you wanted more here?
Can you swap this line to something like:

    defaults=__BACKEND_MODULE % "ItemFilter"

With this a few lines above it:

    __BACKEND_MODULE = "armstrong.core.arm_sections.backends.%s"

I think that'll read better and remove the duplicate in any other backends.
It is still not decided whether we should return a single 5 character code or just 2 character code. If yes, we also need to add
 The country returned is from the country codes defined by ISO 3166-1
 ( http://en.wikipedia.org/wiki/ISO_3166-1 ), using the
 rules set by BCP47 ( http://tools.ietf.org/html/bcp47 ).
to the description.
@hieupham007 This description is not correct. An exception can and will be thrown if the resource cannot be found in either the application or system resource bundle. See my comments above.
You need to qualify TiUIHelper as `org.appcelerator.titanium.util.TiUIHelper`.  I got a warning when generating the javadocs, and adding that qualification fixed it.
Yep just the mock
SIMPLE_OPTIMIZATIONS can be used.
Because this is the directory where the PPS file is.
please put this icon in the repo and rename it to default-icon.png
Why do you need these retries?
It's been some time since I checked this with Tristan but I think he said all service in AS7 are loaded lazily. I will recheck with him, though.
Why the different style for defining the value of UNKNOWN?
I think this should be changed to remove the dependency off blackberry.event. We can call it directly ourselves. Again, probably my mistake :)
Instead of this "collect" function, what do you think about calling `wrench.readdirSyncRecursive(dir);` and then maybe applying a filter to filter .js files. Should make things cleaner.
I have made this change along with the others.
I find the it a bit awkward having the "Older" which brings you to the next page being on the left side  in a left-to-right language but I copied that to be consistent with the diary list page.
Style point: Please don't use `'''` for docblocks nor put the first line of the doc on the same line as the the string character.
sorry to disturb this old pull request I am just testing some github functionality for another project