labofsahil/github-event-dataset-2012 huggingface dataset:
You can't put the logic inside one if clause without duplicating the string concatenation (which is worse than duplicating the if statements) - the if conditions are interspersed.
Would this work if the locales list if empty?
missing getById.
typo: mod_proxy
grammar: purpose-built should be hyphenated
I should have copied the else block :)

```scala
if(POST_LOOKUP)
   body.flatMap(_.get(TOKEN_NAME)).orElse(request.queryString.get(TOKEN_NAME))
else
   request.queryString.get(TOKEN_NAME)
```
需要注意的是，使用index文件会引发内部重定向，请求可能会被其他location处理。
This change breaks backward compatibility.

I don't mind breaking backward compatibility, but if we're going to do that then let's really break it!

Is there a better refactoring we can do here? Are there objects we can extract and hooks we can provide? How can we improve the interface and third-party implementor's experience?
This is resulting in an extra query to the database. It would be preferable to create a new method that find the matching rule and then just use that to get the destination and status code.
shouldnt you return here ?
ok u mean examples such as which include factorial,exp() functions .... may be including these functions in this tutorial is not good ...
you want to do this in `__init__` so that people can modify it if they have to.
Otra tontería sin importancia, pero estaría bueno que podamos mantener el estándar de nombres que llevamos hasta ahora.
Again, mincing meat:

```php
return extension_loaded('curl');
`
to be XHTML 1.1 we need to keep "application/xhtml+xml;"
"Suppose you have *an*" not *and*.
Ah, that's fair.  I guess I was thinking more of a computer sciency way,
like where you have multiple partitions a single disk.  I guess if this
matches the mathematics definition, then let's roll with it.

On Wed, Sep 19, 2012 at 11:22 PM, Keith Stevens <notifications@github.com>wrote:

> In src/main/java/edu/ucla/sspace/clustering/PairedFScoreComparison.java:
>
> > + * NO REPRESENTATIONS OR WARRANTIES OF MERCHANT- ABILITY OR FITNESS FOR ANY
> > + * PARTICULAR PURPOSE OR THAT THE USE OF THE LICENSED SOFTWARE OR DOCUMENTATION
> > + * WILL NOT INFRINGE ANY THIRD PARTY PATENTS, COPYRIGHTS, TRADEMARKS OR OTHER
> > + * RIGHTS.
> > + *
> > + * You should have received a copy of the GNU General Public License
> > + * along with this program. If not, see <http://www.gnu.org/licenses/>.
> > + */
> > +
> > +package edu.ucla.sspace.clustering;
> > +
> > +
> > +/**
> > + * Computes the Paired F-score value between two clustering solutions.  The
> > + * Paired F-Score is similar to the F-Score except that intead of operating over
> > + * single instances, it considers pairings of instances.
>
> Looking at that wiki page, I interpret their definition to match mine. A
> Partition contains multiple parts, in my interface, each cluster is
> equivalent to a part and the Partition contains all the parts. Their
> diagram shows the same kind of interpretation.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/fozziethebeat/S-Space/pull/19/files#r1647778>.
>
>
Symfony ClassLoader component added for ease of use.
Maybe should become a submodule ?
yeah I'm fine with it, but we should maintain it consistently throughout the codebase
Is there a need to delete the kwargs here?
Vladimir, could you move this to a separate commit?
Yes we should. Added a separate issue for this: https://issues.jboss.org/browse/ISPN-2646
I missed that, might work here.
I'll clean this up.

On 23 April 2012 12:27, Sanne Grinovero <
reply@reply.github.com
> wrote:

> > @@ -517,7 +521,14 @@ private Object handleWriteCommand(InvocationContext
> ctx, WriteCommand command, R
> >                       invalidationFuture =
> l1Manager.flushCacheWithSimpleFuture(recipientGenerator.getKeys(),
> >
>      returnValue, ctx.getOrigin(), !(command instanceof RemoveCommand));
> >                       if (sync) {
> > -                     invalidationFuture.get(); // wait for the inval
> command to complete
> > +                     try {
> > +                        invalidationFuture.get(); // wait for the inval
> command to complete
> > +                     } catch (ExecutionException ee) {
> > +                        // Ignore SuspectExceptions - if the node has
> gone away then there is nothing to invalidate anyway.
> > +                        if (!(ee.getCause() instanceof
> SuspectException)) {
>
> same here: -1 to instanceof usage
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/infinispan/infinispan/pull/1078/files#r719773
>



-- 
Manik

Sent from my iPad 4 beta 3 with a telekinetic keyboard
@janl @theophani Please check this is actually correct.
Style: asterisk with the variable name instead of the type: `NSURL *gitDirForLocalURL ...`
Personally, I'm not hard set on 80 characters.  I've seen implementations in the past where that number was derived from the old school computer terminal days.  On the other side of the spectrum, I don't think we should allow lines to just run and run and run.  I feel though, that some magic number for max length of a line would be nice to apply, since we're on the subject of neatness.
This is a bad break.  Just move 'scroll_event' to the next line in the original.
but the former is not pep8 :(

I'll update the PR
Direct ```"None"``` comparison excludes the use of the lowercase ```"none"``` equivalent. I will open a ticket for this.
Apparently not (I always thought it was until this evening):

```
>>> a = u"Hello\u2026"
>>> str(a)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
UnicodeEncodeError: 'ascii' codec can't encode character u'\u2026' in position 5: ordinal not in range(128)
>>> '%s' % a
u'Hello\u2026'
```
Please don't use such vars, it's hard to know what it really is. Something like "dispatcher" is better IMO (In my opinion).
Ah, you can't acutally not rename a file by changing the letters to small ones. you will have to do it in two steps! :) Thanks for catching that.
What about OpenLayers.Util.vendorPrefix? And yes, you are right - this should better be named with a lowercase first letter. To bring it into the Util namespace, just create a new lib/OpenLayers/Util folder and start the vendorPrefix.js file with

    OpenLayers.Util = OpenLayers.Util || {};

If others are in doubt: we also have lib/OpenLayers/Events/buttonclick.js already.
Great, i'll do Rouslan's way.
Thanks.
Found it in forma.schema. It takes timeseries objects a la (schema/timeseries-value 0 [1 2 3]), but all of the other functions are expecting vectors, not these objects, so they'd all need wrappers. I'm inclined to stick with this simple solution, but if you think it's worth working with timeseries objects please let me know. I could be missing the point!
Should say `(e.g. sites widget)`
Sensu 0.9.6 changes $settings to a hash, so this will need to be updated :) $settings[:dashboard][:autorefresh]
Hi, thank you, I have fixed the other two suggestions. But concerning replacing std::vector with SGVector, I do have some problems here.

Actually, I don't know how the SGVector object should be used. First of all, it is not ref-counted, so we should use it as ordinary containers. However, I'm a little confused about its design. For example, it sometimes allocate the memory for itself, and sometimes simply re-use the memory from the outside or another SGVector object. But in neither case, it do not release the memory at destructors. We have to call a function explicitly to release the memory, and should be very careful about where the memory comes from for each particular SGVector. So I'm very hesitate here to use SGVector, I'm wondering whether there is any conventions on how this object should be used to avoid common memory errors? If there's any, it would be very helpful for me (and potentially other developers). 

BTW: I also saw at the ideas page that SGVector is going to be re-designed with ref-counting support. I would be very interesting to know, if any of you could explain something about why it is designed this way, and in what way do you want the future SGVector looks like? Thank you!
Haha, that must be the most used number in programming doc history ;)
The session argument to this function is conn->default_session, AFAICT -- why are we using it explicitly throughout this function (and the comment doesn't make sense to me for that reason).
Should the type for parameter ``$locale`` not be ``string|null``?
统计markdown格式的就好了。.md或者.markdown后缀
Was passiert, wenns mehrere REQ_ATTRs gibt?
Can we change this too: 'Retrieve the render array representation for the given block.' please?
if you want to get the original class name, you should probably use ``Doctrine\Common\Util\ClassUtils``
please use the opposite condition and throw the exception, which then avoids uisng ``else`` for the other case
I do not understand why you wiped whole the `.gitignore` file. With this I cannot even theoretically accept the pull request.
Wondered about that. Good.
Missed this one before. You don't need the run_command here. `http_client_get_url` is just poorly named. name. It should be just `http_get` or something since it already runs stream_command internally.
Não precisamos de `ticket_id` no servidor: uma Activity tem vários Tickets (`has many :tickets`).
I only used sets so I could intersect them. I didn't realize that the ordering was based on the order that setAttribute was called. I assumed that was dictated by the feature type.

Since we won't have a large number of attributes here, I don't have any problems with using lists instead.
Eventually it will be used elsewhere, I'm sure. Renamed to createTimestamp and set to protected.
If this file is empty now, It'll be better to delete it, what do you think?
Are you asking this for this specific function or is this a general question why all functions are only referenced in the API?

My main concern with the old implementation was that you couldn't see quickly which functions were public and which not. That's why I grouped the API in the return statement. There are two options:

- either we reference all public function as I do in this pull request
- or we put the public functions in the `return` statement.

Opinions?
I'd love to use "puppet config --mode master print confdir" instead of assuming "/etc/puppet" but unfortunately the proxy is running as non-root, so you get "/usr/share/foreman-proxy/.puppet" back....
Hm, we might move this to the method of node class?

I believe we store backend information in the node class (we inject from outside, when we crate it). If not, we should. :-)

So node class should have information from which backend it is. And then this could be method on the class.
I am a bit skeptical about this function. It looks strange. And the query above. But OK, leave it as it is. Or maybe add TODO saying to check and possibility optimize.
Should I also do that for facebook backend?
I chose this approach because I didn't wanted to define all resource fields manually. Will change it.
It is strange that you should be doing this here. This should be done in the function which fetches the content.

Also, Django has already some functions to handle such things. Try searching in `django.utils`.
I don't understand this. So how many times they make model prediction per day? How many values each prediction has for different time spans? How man time spans are for each prediction?
What is this? Why not just:

```
ngettext("%s day ago", "%s days ago", days)
```
Goh. Comma at the end of line. :)
`isAutoShowIncoming` -> `autoShowIncomingPosts`.
I'm amused by the "far too accurate" comment.
This is expected to happen only if `space` is not an empty string. `option.format.compact` results in `space` becoming an empty string.
Yep, I broke my rule and didn't try compiling it after the change.
I wonder if we can think of a better name for 'week_start' ?

I'm terrible at naming things, but "week_start" sounds like a datetime, but we're trying to indicate "Did the user click 'update' on the date selector?"

Perhaps update_week?
Please remove the default from the XSD
We do not need a test for every month, one example is perfectly fine (just test `Jänner`)
yes it's an example even though quorun is a shared concern and might be left globally.
I think it would be a good idea to register the tag under 'cite_details' – we can keep 'cite_d' as an alias though.
I believe in this case the build is required. It's not a python build directory. That's where the swf is built (same in mojombo/clippy).
I don't understand why we have parse_env_str and parse_env here. Any explanation?
See the beginning of `window.templates.resource_detail.update` ( around the line 230 ):  This template is only called if the corresponding `resourceInfo` object has a `data` property ( i.e. the network entry has a `responsebody` )
In the header filter (ngx_http_header_filter_module.c), the header hash value with 1 is added in to the response HTTP, otherwise, it's ignored.
should we check that this is actually an integer 
if not k == dfdb.shape[1] / (J-1.): raise ValueError('something fishy, maybe no constant')
хм. то что написано. проверить что UploadCare raises
не знаю. дай подумать/обсудить
Please keep spacing the same as subsequent lines.
but to your point, this code is also somewhat needlessly terse, i should fix that so it's obvious what's going on
You should populate your alt attribute here dude, otherwise you might lose point on your accessibility tests
Could we imagine a circumstance where the requirement that the ```PhotonArray```s have the same size would be problematic?  An example I could think of would be if we have a complicated galaxy requiring tons of photons to represent properly, and a fairly simple PSF that is easily represented with a factor of 10 fewer.  Do you think this is unlikely to happen much in practice?
Errr, I did that before I saw your comment, Barney.  I had been thinking we'd be able to use the input to make the SBProfiles with their parameters properly set.    Hmmm.
nitpicker's corner: you want an AssertionError for any code that's unreachable.
well thats from the original code and technically not mine :) ill make the change though.
Sure, I understand false positives aren't tragic, could just be a
mysterious tough-to-debug performance hit if/when it happens.  I'm OK with
this.
I didn't realizing we were deprecating editing a campaign...

How is editing supposed to work?
not that i know of... this feels right for now
Should it be invalid (e.g. throw exception) if an `ssl != null`, but everything is null (e.g. sourceCertificateUrl/sourceKeyUrl are null so not copying, and certificateDestination/keyDestination are null so no pre-existing files exist.
I'd push this version definition up to the top-level pom.xml. That's where all our other version dependencies are defined, so good to keep them together.
Looks dangerous, swallowing an exception without any logging
I will rewrite 0 to null.
This file should not be included in the repository
cursor->opts |= link->servers->read_pref.type != MONGO_RP_PRIMARY ? CURSOR_FLAG_SLAVE_OKAY : 0; ?
а запятые корректно обрабатываются методов to_f ?
Do we use gyp for Android? Since we do load the module in windows, we can provide the dummy implementation for the time being and have it working for linux since the work is done.
Well, it IS really slower, esp when you have several hundred things in both lists.    I'm so familiar with this sort of code that it seems "clean" to me but I understand how it might be weird looking.  I'll add some tests.
WEll, it would more accurately reflect the structure of the data somehow;  it would mean that one table would be "job management data" and would not be edited by the bandit.evaluate, and all modifications to it controlled by the bandit algo and Experiment, and the other table would be "results data", with the possibility of having many result records associated to a single job record, which would be written to asynchronously by the bandit.evaluate.
I think `dar` should be `dat`, no?
Since ```page``` is public, this method probably isn't needed.
Should we keep this here or move it to a controller which handles Friendship?
Just a note, this is another good place to use the guard-style as noted in my other comment about UserRole#approved?
La ce afisare te referi ?
Remove.  Should not be exported.
add_column fails. not update
Это некорректно. Любой объект можно логировать.
Для этого есть метод length()
Так табуляция запрещена на Code Conventions. Только 4 пробела.
Ещё бы как-то это отображать. Хотя бы в меню.
This line must be returned by the patches method. Read the comment above it.
Suggestion: provide a docstring for this method as well as <code>name</code> and <code>state</code>
Required:  Use double quotes (just to be consistent).
did you set the autocrlf option in git?
http://wiki.xbmc.org/index.php?title=Git_Usage_Windows#Line_Endings
Maybe rename folder to 'demo'?
Agreed with @larsmans.
noted.. thanks :v:
Is this really a pseudolikelihood?
Can you give a reference where this  was used?
can you add that it is returned only if return_path=True?
I actually looked it up and ``dependant`` seems to be possible in British English but it looked weird to me.
it seems more natural to have clf.named_steps['reduce_dim'] return the PCA instance. WDYT?
On 10/09/2012 07:17, serge-sans-paille wrote:
>
> In pythran/pythonic++/core/set.h:
>
> > @@ -106,15 +128,233 @@
> >  
> >                  template<class U> // just for type inference, should never been instantiated
> >                      set<decltype(std::declval<T>()+std::declval<U>())> operator+(set<U> const &);
> > +
> > +		// Misc
> > +
> > +		set<T> copy() const{
> > +			return set<T>(*this);
>
> What the hell ? Didn't we expect set(begin(), end()); ?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/serge-sans-paille/pythran/pull/12/files#r1563237>.
>
My bad i got confused in my branches...
If you want to access the `Room` instance here (which is known to the lamp) then you need to pass the room to the new bulb in the `Lamp` initializer:

```ruby
@bulb = Bulb.new(room)
```

BUT

This is also a good example of how in "real-life" programming we think a lot about which class needs to depend on which other class. If the sole reason why you need to access the room in this place is so you can output a message then maybe it's sufficient if the lamp does that for you. Because the Lamp is the thing where the outer world operates on (using `flip_switch`) whereas the `Switch` and `Bulb` classes could be more like internal of the `Lamp`?

These things are entirely up to you as a developer, and in your example I guess it would be just fine to just make the `Bulb` know which room it is in. But imagine that you have a lamp (which has a bulb) and it can be moved (carried over) from one room to another. Now you'd need to make sure that the new room get set on both the lamp *and* the bulb. This is why you'd probably not want to make the bulb know the the room, too.

Another, although more complicated way to solve this is to make the bulb only know the lamp and then call back on the lamp when it switches on. But that seems more artificial in your example :)

Why did you go from `on` to `light_comes_on` as a method name?
Oh, I thought it defaulted to private.
The order in which attributes are written should match the order in which they appear in the xsd.
To be honest, I have no idea why it is in there, so I just didn't touch it. Speculation mode on: Allocating `size_t.max / 3` bytes might be possible on some 32 bit systems if they provide an uncluttered address space to applications, but maybe this is guaranteed to not happen on Windows?
Also disable the copy ctor and assignment operator
After refactoring the class and wrapping all controls in containers, getNativeHandle was no longer required. The handle to the Cascades control is private to the native class that represents it and the properties and methods that require access to the control override the methods in the NativeControl base class. If it's required in the future, it can easily be re-added.
I'm wondering how far this is really suited as a boolean attribute. Would it not make sense to assume that someone might want to introduce a better strategy than the two alternative strategies we have today? Should we not use an Enum ?

Also I don't think I understood the javadocs on these options. That is likely my own limitation not being familiar with spatial queries, still it would be helpful to elaborate a bit here as it seems I need to keep a copy of the reference docs handy to understand this.
minor: I'd have moved it up together with other attributes...
Even if it's no dataset? Shouldn't it have to ```(value & childdataset_dsmask != 0)```?
Good point, changed back to realloc
Should this be also supplying max?
>By "number of spill files" I really meant the "number of occasions rows were written to each spill file".

yes, and I'm using the # records written + peak total in mem overall to estimate # buckets on rounds >= 2
If an exception is thrown in ```useINotify```, you close the queue, ignore the exception and try to find the differences anyway?

I think you should at least return NULL, but better to just propagates the exception upstream and let the caller deal with it, if possible.
you should remove this commented code
ack, yes. it's a typo. I'll push a fix into the branch. thanks for catching that.
@thatothermitch this is a perfect example of copying and pasting something without really diving into it.  I stole this from: https://github.com/sinatra/sinatra/blob/master/Gemfile.  I agree that the sub line is unnecessary and is probably a hold over from some older API they had.  I'll remove it.
This method should be renamed, but other than that this looks awesome!
This is totally useless. It will never be skipped here as ``self::$skipped`` will never be set. Please remove this stuff
This line should be:                    
var circle = paper.circle(cx, cy, r).attr({ fill: opts.colors && opts.colors[i] || chartinst.colors[i], stroke: opts.stroke || "#fff", "stroke-width": opts.strokewidth == null ? 1 : opts.strokewidth });

Otherwise you're not getting the colors set up in options.
csn we use self.timeout here too?
basically nice, I'm only curious if it wouldn't make sense packing the sourceModel and source object, as well as the targetModel and target object into separate objects of the same type; e.g. model-object-coupling...
This is fine, but I'm slowly starting to adopt ``cimport numpy as cnp`` to distinguish between the two imports.
While I know this is contrary to PEP08, I really prefer ``2**degree`` above ``2 ** degree``.  At a minimum, at least ``(2 ** degree)``.
I suggest "Random walker algorithm for segmentation from markers, for gray-valued or multichannel images"
These plots all look very similar... is that expected?
+1 on removing this parameter.
common.instance_exists already catches the nova_exception and raises exception.NotFound(), no need to duplicate it here
As previously discussed, no it's not wrong; it's here mainly for consistency and self-document-ability of code.
Okay, I'll make both changes.
Given that the water material provides sensible defaults, should we really show all these uniforms here?  Perhaps show just the ones that are most likely to be changed?
It used to return the position in the camera's reference frame. I will add that the order of the arguments changed and code snippets.
Unfortunately, the HTML specification doesn't list or require any specific formats, its up to each browser to decide which formats are worth supporting.  Because of this it's impossible for us to determine which formats are supported, either ahead of time or at run-time.
This is just for convenience so simple_form won't wine about the method remember_me not existing on User.
Non-functional statement. Should this be the following?

```JavaScript
jqplotRef = $.jqplot;
```

Also, group vars.
We should leave a comment here on what we're doing here, like all other setup stuff we do in this method.
This should probably be "height", not "width".
Android also has this property.
Are these available at construction time only? What happens if the dev does:

var a = Ti.UI.createAlertDialog({blah})
a.messageid = blah

Won't it get ignored?
Ok, so I will change all shortcuts to its bind respective.
I don't get how this was better than it was before.  Creating a new function, only to use it one time seems more complicated.
we don't need to have a way to run something related to a tab when the widget is just created

however, we need to have a way to run something related to it when the dialog is opened

so it's natural to construct tabs when the dialog is opened, and not earlier
I'm more excited about the `method_added` approach :)
https://github.com/wojtekmach/minitest-metadata/pull/3
For some reason, I am getting formatting I don't expect. Guess I will have to learn more about the comment syntax.
yeah .. i guess so @cordoval can you also send a PR to phpcr/phpcr to require phpcr/phpcr-implementation ?
Indentation error here.
Make this just waitTime, and we'll put in the docs that it's milliseconds.
Ouch. You're right. Sorry about that.
That's terrible, terrible. I hate this makefile and everything related to it.
appender!X is initialized, Appender!X isn't. Appender will lazily initialize itself, so for most code it there isn't a difference. However, not all code that takes an output range uses auto ref. In those cases the original appender is never initialized, resulting in the appended data never being visible from the source location. I could alter the ctors to allow

   auto app = Appender!X();

as an equivalent construct. Would that be preferable?
Please add original test case that is in bugzilla.
Can you add @Deprecated over the method?
i was in the mood and forgot to add it as i did my crazy dog!
Include

Copyright 2009-2012 Ghent University
We can also consider pass our function as a handler to emit.
maybe move the Object array into the if block?
no shadow?
??? Duplicate of line above?
Boolea**n** the `n` is missing
Nit: I see this same string all over the place. Would be nice to refactor into a constant.
ah .. might be a new feature.
Properties are always cooler than methods. We are using C# after all :)
Does this support using atom:link and sets the namespace?
Unfortunately, I think I need a for loop, rather than this cool dict / list stuff, because if the object doesn't have a name, I need to skip and warn.  So replacing it with a for loop.
Small comment error - should be

If the widget was floating __right__, add it to the right column
Probably a String or Integer
@croby Thanks
Use the block syntax instead just so the file handle is properly closed if there's an exception, like so:

```ruby
File.open path do |ios|
    ios.write ...
end
```

Also why are you opening the file to append to it? Shouldn't we be replacing the contents?
Here's how to do it wrong:

https://github.com/ivmai/bdwgc/blob/master/os_dep.c
Instead of the `if … else` please use the `:authenticate_citizen!` before filter.

`#fetch_objects` should also be a private method.

See the `CommentsController` as an example.
We should add a space before '('
Only combination with the require-dev part i found that worked
I had performance issues serializing the data. This seems to be the fastest implementation.

On 29. 09. 12, at 12:04, Mitar <notifications@github.com> wrote:

> In django_datastream/serializers.py:
> 
> > @@ -0,0 +1,18 @@
> > +import ujson
> Why is this one better?
> 
> —
> Reply to this email directly or view it on GitHub.
> 
>
лишнее «и» в перед «PHP 5.2»
This wasn't changed before?, anyway Ok
No need to keep my previous lame commenting style.  I'd remove the blank `//` lines throughout.  Less is more.
像这种redirect行为应该留在view.php中
用正则处理，尽力扩大灵活性，比如能适应域名改变什么的
Should this be translatable? How it is in official fields?

Probably *ID field* would be better description?
No, I never think it's `unittest.TestCase`, though `DjangoTestCase` would be better. Regardless, there are two definitions of `TestCase` that we're stuck with. This pattern makes it at least four in each project. `ArmSectionsTestCase` is very clear and not particularly onerous to type. I don't see how this change is an improvement.
don't agree on that. the relative path should always point to your full path. not just the get params. 

e.g. with /api/photos/?foo=bar i will get much more valuable information and can combine it with get_hostname for a full url. also for mobile / other api clients it's much easier to use
With the use of nullable types, the above two lines would become:

    if (Timeout.HasValue)
    {
        writer.WriteProperty("timeout", Timeout.Value);
    }
Java championne a mettre en gras :-) C'est presque un peu tard dans le post, genre c'est un détail, alors que ca me semble une annonce assez importante!
I not removed PHP magic functions. Review properly.
Your username should read "(ledermueller)" as it is on github:)
Ya, that is fine by me.
The interface does not guarantee, though all objects passed do create the transaction. 

I'll revert this change and submit the assert of all similar cases in a separate patch.
Would it make more sense to 'create DeployableContainer / ContainerConfiguration classes' then talk about how Arquillian picks them up via the LoadableExtension registration?

At this point you're registrering classes that does not yet exist..
Use the logger to log this. Print stack trace just goes to sdout.
Found it on some web site when searching about progress bars, quite neat, can highlight the important bits - good as there are quite some numbers in that bar now. It's using ANSI codes, only works if your terminal handles ANSI (like most if not all modern terminal emulators will, I'm sure).
We don't like System.out :)
why does the priority need to be higher than 10?
Same as:
~~~clojure
(my-map first seqs)
~~~
Those vars are garbage we forgot to clean from other code versions... its ok, it was very late to think properly :D
Which case is his line trying to handle?
I designed it so that domains could be added before webviews if we ever decide to go that route. This is to avoid adding a domain that is in the config and having duplicates or incorrectly updated behaviour.
what does this page do?
If its just a test, we should move it to the appropriate folder, this doesn't seem like the best place for test pages. Functional tests maybe?
should check this against the latest trunk if you are not seeing an empty response in R9
It's build time so no OSS needed
test that when you grab supportedSensors twice, it only calls execSync once
Fantastic. However after putting back line 22, it works as it should everywhere else, but it's also outputting the string "http://fluidstudios.local/wp-content/themes/fluid-studios" where that line of PHP is.
Indentation error & capitalize first letter filename
I didn't really remove these lines, I just spaced them out a bit so the equal signs would line up.
That's just the kinda guy I am.
Last thing  `setCondition(array $condition)`