labofsahil/github-event-dataset-2012 huggingface dataset:
This test isn't entirely sufficient. We should test the following steps:

 1. Creating a subscribable on a background queue.
 1. Not adding any subscribers.
 1. Still on the background thread, wait a second or so (any sufficient amount of time for the main run loop to spin at least a few times).
 1. Verify that the subscribable hasn't been deallocated.

I expect that this will fail with the current implementation.
:+1: I'll clarify the docs that they'll be called immediately after the current block.
@codyhoag I recommend putting these action related instructions just after Figure 8.14 "OpenSocial Gadget Publisher" as it is convenient for the user to work with the actions just after publishing the gadget.
No need an import for that.
Bad i18n. Incomplete sentence.
why it looks weird @stof ?
删掉无用空行
因为会影响生成的页面
缓冲到达一定<value>数量</value>时
->
写缓冲到达一定<value>size</value>时，nginx就会向客户端发送响应
Why don't you use a simple `{{include}}`?
I think we could go with the first option and migrate to the second when the template information will be available.

So AFAIU we will need to add to the wiki object in the model the following information:

* Owner
* Pretty name
* Description
После ретурна исполнение дойдет до последней строчки метода? ЛОМАЮЩИЕ НОВОСТИ.
не работает лямбдой. Поэтому и сделал функтором. А имя забыл поменять
El tema es así: Raven convierte 

    x.FileNameWithoutSpaces == Parameters.Term

en algo así:

    FileNameWithoutSpaces: TermValueEscapado

entonces usar comodines no funciona, pero con StarsWith parece que es distinto:

    x.FileNameWithoutSpaces.StarsWith(Parameters.Term)

en algo así:

        FileNameWithoutSpaces: TermValueSinEscapar*

Huele a bug, luego vamos a buscar una mejor forma de hacer esto, también el tema de los espacios y los acentos (estos últimos no los estamos resolviendo bien)
Me parece que no
Could this be integrate_prepare_message_context, for clarity?
Remove $response requireNotGuest Throws an Exception
What will mapMode be if the caller forgot to set it? Should we check/log/handle that?
I think you forget modify code for CCPhysicsDebugNode.js : 
  this.drawPolyWithVerts(shape.tVerts, shape.getNumVerts(), color, 1.0, line);
"try" seems like a fun method. Where is it defined?
This should use `<%= node['mongodb']['port'] %>` rather than hard-coding 27017
For this test, I won't change much, but it's good to have a mix of tests. It's a bit like old config vs new config. New tests are written using new config. Not too fuzzed about this but just making sure we increase the protocol coverage.
As I posted on the mailing list, for that I'd need XMLSchema 1.1 which adds support for <xs:any /> within <xs:all /> elements. Unfortunately all major Java IDEs don't support XMLSchema 1.1 (as it was ratified in April 2012), and having a 1.1 schema would be counterproductive. We can definitely move to 1.1 if the IDE situation changes by the time 6.0 comes along.
Vladimir, how about only extracting the root cause if the current exception is an ExecutionException?
If the user code throws a `new ApplicationException(new NullPointerException())`, I think we shouldn't discard the ApplicationException.
And I still don't see why the data container needs access to the
interceptor - which is definitely a no go.


On 10 December 2012 16:26, Manik Surtani <manik@surtani.org> wrote:

> I don't follow you at all.  What has eviction (segment size exceeded) have
> to do with this?
>
> 1.  When a segment size is exceeded, the item should be removed from the
> data container (this happens internally, NOT by calling
> datacontainer.remove()) and it should NOT be removed by the cache store -
> and has nothing to do with the activation count.
>
> 2.  When an entry expires, it is removed from both the data container and
> the cache store. Again, this has nothing to do with the activation count.
>
> 3.  Activation only happens when the ActivationInterceptor can't find
> something in the data container and has to load this from the cache store.
>  This is the only place where activation counts come in.
>
>
>
>
> On 10 December 2012 16:16, Galder Zamarreño <notifications@github.com>wrote:
>
>> In
>> core/src/main/java/org/infinispan/util/concurrent/BoundedConcurrentHashMap.java:
>>
>> > @@ -1714,6 +1723,14 @@ V remove(Object key, int hash, Object value) {
>> >                    count = c; // write-volatile
>> >                 }
>> >              }
>> > +
>> > +            if (!isEvict) {
>> > +               // If removing (and not evicting), activation rules
>> > +               // require entry to be removed from cache store (via
>> > +               // activation) and counter to be updated
>> > +               evictionListener.onEntryActivated(key);
>>
>> Manik, if the removal is normal removal (i.e. cache.remove()), it's all
>> fine, but there can be a removal triggered from within the container (i.e.
>> an entry has expired, the segment size has exceeded...etc), so to be able
>> to update that counter and if you wanna keep the counter in the
>> interceptor, the data container must have access to the interceptor.
>> There's not a single way in which data can be removed.
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/infinispan/infinispan/pull/1455/files#r2365462>.
>>
>>
>
>
>
> --
> Manik
>
> "Physics is like sex. Sure, it may give some practical results, but
> that's not why we do it." - Richard P. Feynman
>
>
>


-- 
Manik

"Physics is like sex. Sure, it may give some practical results, but that's
not why we do it." - Richard P. Feynman
I think it would be simpler if you leave this line here and compute the remove flag after the (successful) unlock.
I would just return immediately here:

```
if ( ! is_file($file)) {
    return time();
}
```

We do not really need to traverse the other paths at that point.

Maybe you can also add some tests to the FileAsset directly, should be pretty easy.
Why "old tree?" Couldn't this tree be "newer" (whatever that means in git anyways) than the index?
Yep, I was thinking of the problem from the completely wrong direction.
There's also a statsd output which also has a StatsdWriter. This will prolly happen a lot. Names are hard. :(
Move this to the test rather than the setup
That's fixed, thanks for the review.
Don't override this base-level element.  Use class="clean" on `<ul>` elements that you don't want to have bullets.
Just noticed this should be 'delete[] allowed'.
Ray - agreed on all points. These are the refactoring steps (or something similar) that I was alluding to in the description of the PR. Rather than Query queryConfigure (Request) I was thinking something along the design I presented earlier where a QueryFactory contains a Query getQuery() method. The actual search service would need only that QueryFactory to execute a search. All of the messy details of PropertyProvider, ResultWriter, and dealing with SlingHttpServletRequest can be shoved into a "legacy" QueryFactory to preserve backward compatibility. New designs for managing query construction could be imbued in a new QueryFactory implementation.
maybe this should be something like object.repository.permissions.where(:admin => true).try(:user) ?
The FALLTHROUGH doesn't make much sense to me, we're falling through a case where nothing applies.   I'd switch to a go-to.
Should this be "create a named checkpoint"?   I realize it's snapshots underneath, but there's nothing here to explain why you might name the checkpoint's snapshot?   (It's early days in the review, yet, so this comment may be completely off-base.)

There's a problem with the session->checkpoint function description, which says "Flush the cache and/or the log and optionally archive log files.", and you've removed the configuration strings that flushed the cache and/or the log.

The docs for these APIs and the example code (ex_transaction.c) still say the APIs aren't  supported in WT.

The session->commit_transaction and session->rollback_transaction docs say "Ignored if no transaction is in progress", which doesn't match the code which returns EINVAL.  The session->begin_transaction docs say "Ignored if a transaction is in progress", also doesn't match the code.  I don't see any reason to ignore, off-hand, I'd say the code is correct.

The session->commit_transaction and session->rollback_transaction docs say "Any cursors opened during the transaction will be closed before the commit is processed", session->open_cursor says "The cursor must be closed before the transaction can end. If WT_SESSION::commit_transaction or WT_SESSION::rollback_transaction are called with cursors open in the transaction, the cursor handles will be closed implicitly and must not be accessed again."; I don't see where that happens.

It's a little odd that you can open a cursor outside of a transaction, and perform operations, but then when you begin a transaction cursors are suddenly integrated into those transactions, but it makes sense.   It would seem logical to me that if cursors simply appear in whatever transaction the session has at any time, that is, if you don't have to explicitly open the cursor "inside" of the transaction, then committing or aborting the transaction wouldn't affect the cursor, either, only the operations previously done using that cursor.

What if an application opens a cursor, but because of a coding error, it updates a record before starting the transaction.  In a transactional setting, it could be very bad.   Should we defend against that?
In other template files this is wrapped in its own container, usually a table cell. Please do that here too, wrap it in div or extra table. See viewtopic_body.html for example.
looks like the rest of the file is in alphabetical order. might want to move this above /phpBB/files/* ?
For now I've changed it to use fmt in 49e90b255bf72e77e5ed94ddb318db0d388505ed.
Also, otherwise, what's the point of having an interface if we don't require them to implement it?
changed code to this
size.text = (isBlackberry) ? l2.getHeight() + ' ' + l2.getWidth() : 
										l2.size.height + ' ' + l2.size.width;
wird aber eh benötigt, wiel der helper in der applikation registriert wird.
starting from here
Кстати, серьезно, не лень было в каждой функции все это прописывать?
I wonder if we could move these constants out into a seperate object so they are only defined in one place and write a merge function to copy the properties over, so that this code would look like

this._structured_clone = merge({name:String(this.name), status:this.status, message:message}, test_statuses)

and something similar when they are put on the Test prototype object.
It has been called body from the start. Maybe they were missed by the switch to JHttp.
Estou a pensar no config/locales, mas não estou a ver muito bem como.
Ops, sorry about that :)
This documentation is a little confusing - perhaps it should read "fired" from the mark_import_pending task?

It should also include some information about what the listeners should return. From the task, it looks like they are meant to return dicts or Q objects to be used to filter the queryset.

Also, this should be done with ``#:`` - see also [Sphinx autodoc docs](http://sphinx.pocoo.org/ext/autodoc.html#directive-autofunction).
ditto re commenting out?
instance_id_t вроде типа uint64_t, а не size_t
Again, could we put a little comment in about these for ignorant cusses like me?
haha `.end()` took a while to grok, partly because there are native 'req' and 'res' and also Request and Response and it's easy to forget which one you're working on. I'd love to split out some of those callbacks in `.end()` to named functions somewhere else.
Actually I found that auth.User already has method get_full_name. Will use it instead.
Probably no default is needed. Or check how we configure gender elsewhere.
Aaa. Noo. What, we had datetime.now??? How I missed this.

The solution should be:

```
from django.utils import timezone

timezone.now()
```

You do not have to complicate with replace of timezone.
JavaScript has `unread_counter++`.
This is wrong use of `blocktrans`, read documentation. ;-)
(This is not Java, that everything has to be in classes.)
Because this is not really a class method, but a static method.
Use `self.error_messages`. Maybe you could still values for it from `UserCreationForm`.
*varno* -> *brez skrbi*
Oh, this will sooo blow up some day. ;-) Because if frontend would import anything from Django, which would need settings, which ...
why not ```test -n "$SPID"``` negation is not needed ( or shortcut test "$SPID" )
Nit: I know it looks nice but PEP8 explicitly says don't line things up like this ;)
These headers from membase's memcached and they contain additional commands, which isn't defined in vanilla memcached. And the url points to the directory. Actually it is just prefix for two headers we need.: `https://raw.github.com/membase/memcached/engine/include/memcached/protocol_binary.h` and `https://raw.github.com/membase/memcached/engine/include/memcached/vbucket.h`.

It uses curl or wget to download them
Aquí hay un detalle y es que previamente debe de existir el directorio `${basedir}/conf/` lo que hice fue crear el directorio y volver ejecutar el guión el cual se ejecuto correctamente.

Me queda mi duda si no es mejor idea el crear este directorio dentro de la función `create_structure` la cual quedaría como:

```sh
create_structure (){
	echo "-> Creating '${dest}' structure"
	(mkdir -p ${dest}/var && mkdir -p ${dest}/bin && mkdir -p ${dest}/htdocs && mkdir -p ${basedir}/conf )
	if [ ${?} != 0 ]; then
		echo "\n-> Error while creating the structure"
		exit 1
	fi
}
```

¿Que opinan?, igual y estaría bien realizar el cambio antes de cualquier merge.
Since this is the owner's id and not the CMUser instance for the owner itself, can we name this `ownerId`? That way it'll be easier to deprecate without breaking backwards compatibility when we do add an owner field that references a CMUser.

Better yet, can this just be the CMUser itself? I forget if we talked about a reason why this had to remain an ID.
Should be left as is because it is a module name and not a model name.
This applies also for the other classes with private properties?
In this particular case, I preferred the alias, as we don't know if another CLI application will be used
the fix is already committed by @repeatedly :+1:
I think you should avoid referring to the work on Voldemort, as I don't think it adds much value in the javadoc.

You can also use private comments if you would like to better explain some code occasionally (would be nice!).

On the voldemort code reuse.. if it really is a copy&paste work, do you think (after they are both integrated) we would be able to move common routines in the hibernate-ogm-core module as a general utility library?
Wrong copy/ paste.
That string actually applies to the `view-parsed` setting.
Other than this "incomplete" switch between **raw** and **parsed**, it looks fine.
Yea, removing the checks.
nginx already does this, right? If it's required, use sigaction(). That's what nginx uses and signal() and sigaction() don't necessarily mix (blame POSIX).
BTW, float(M_PI) is a "C-style" cast, and should be avoided unless really necessary (ref: Scott Meyer's "Effective C++" book)
This is the simplest approximation that can be made for quantiles. Not sure if the name isn't misleading; for getting actual quantiles something like linear interpolation is better. That's not needed here because all you do with it is plot them. Maybe a name like sorted_data would be more accurate.
Oops, I can just use `block_data` inside the block, right? Sorry!
I guess `Hash#fetch` works well for this case, then.
Why parenthesis here?
Isn't that match too greedy? I don't know if it's applicable here but consider this:

```javascript
> /(<a.*a>)/gi.test('<a href="foo"><script>alert("evil code here")</script></a>')
true
```

Then there's the whole philosophical debate of parsing XML / HTML with regular expressions, of course.
Quite right!  Error fixed and pushed.
On Jun 13, 2012, at 11:49 AM, Mike Jarvis wrote:

> Or use ctrl-v to get a literal tab.  grep '[ctrl-v][tab]'

On Jun 13, 2012, at 11:52 AM, Rachel Mandelbaum wrote:

> Mike, if I do that I get tons of things that are actually a sequence of spaces... ?

Weird.  Maybe it's OS-dependent.  I only get things that are tabs.
Geen `?` aan het einde?
You can combine this folder with /var/log/zend like this: https://github.com/TrafeX/puppet-project/blob/master/modules/project/manifests/web.pp#L7
this was deliberate.  i want to make the output look nice, so if there is a message it is "nice", otherwise we show standard message.  original error is included in the trace so we don't lose the not-nice info, we just de-emphasise it.
has been amazing (present continu quand tu parles d'un processus etale dans le temps et  qui n'est pas termine)
이것두 Play 마켓으로 변경할께요.
Can you also put this in `scripts/`?
in the future --> into the future (?)

Maybe refine the first sentence; something like: "In the future, the add-on store could provide couple of alternative Blog Add-Ons to choose from for this kind of situation."
rainy day ponder if there's a way to have inheritance here, as seems this is a choice object where only one of the elements would be present
Conform Code Conventions este recomanda sa fie doar o declaratie pe line.
Okay =)
If you have other ideas for gouge, you are welcome, i still don't understand how effect 2 coult be broken by effect 0.
And in unit.cpp, i forgot a line, I added it as a comment.
-1 on passing this `GsonBuilder` directly, **if** we need to change this signature, let's pass the config object then
s/httpProviderProvider/httpProvider? It also could be final
Второй раз создаётся new File(destination)
This is not correct. `actualTest1` should be equal to "\u0005". The XML should be un-encoded when read back from the server. If it is not, then you have a bug and the test should be failing.
I'm not sure @Marlena will like these kind of loop and logics inside a test without a good comment! 

It's quite difficult to read.

The way I see it is the two tasks this is trying to achieve are basically `deselect_all_devices` and `check_device(string)`. Could these be made into two methods to give readability from the test side?
あー、これはキメの問題だと思います。個人的に日本語では文末：は使いませんが、grepしてみたらこれまで使ってきているみたいなので、コロンいれときます。
This should be in English with the `{% trans %}` template tag.
Likely not content of this PR but do we need to consider try/finally's for these streams?
Переименую.
If this isn't needed, should remove it...
This exception is not tested.
Do you think approximating the normalization with a `randomized_svd` with an apriori fixed rank would speedup the computation without causing an accuracy drop?
I'm having a little trouble following the code -- but it looks like if there are folds, then actually different random hyper-parameters will be used for each fold.  I'd recommend against this, I think it's more meaningful to compare between folds of K-fold validation if the same set of hyperparameters is used on each fold.

Apologies if that's not what's going on here, but I wanted to toss that thought in.
You can look at my branch arithmetic for that.
I had to modify NumberTraits.h and NumberTraits.cpp

On 06/03/2012 11:10, David Coeurjolly wrote:
>> @@ -70,7 +70,7 @@ inline
>>   DGtal::PointVector<dim, TComponent>::PointVector()
>>   {
>>     for ( Dimension i = 0; i<  dim; ++i )
>> -    myArray[ i ] = 0;
>> +    myArray[ i ] = NumberTraits<TComponent>::zero();
>
>
> I did that but got weird results for GMP. I'll investigate..
>
>
> Le 6 mars 2012  11:06, Jacques-Olivier Lachaud a crit :
>
>>> @@ -70,7 +70,7 @@ inline
>>> DGtal::PointVector<dim, TComponent>::PointVector()
>>> {
>>>    for ( Dimension i = 0; i<  dim; ++i )
>>> -    myArray[ i ] = 0;
>>> +    myArray[ i ] = NumberTraits<TComponent>::zero();
>>
>> myArray[ i ] = NumberTraits<TComponent>::ZERO; // faster ?
>>
>> ---
>> Reply to this email directly or view it on GitHub:
>> https://github.com/DGtal-team/DGtal/pull/293/files#r522346
>
> ----------------
> David Coeurjolly - Directeur de recherche CNRS
> CNRS, LIRIS-UMR 5205
> Universit de Lyon
> Batiment Nautibus
> 43 boulevard du 11 novembre 1918, 69622 Villeurbanne cedex, France
> Tel : (+33) [0]4.72.44.82.40  Fax : (+33) [0]4.72.43.15.36
> http://liris.cnrs.fr/david.coeurjolly
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/DGtal-team/DGtal/pull/293/files#r522352
These should be defined in \<avr/io.h\> as PIND0 and PIND1
I'm fully aware of the problem, what I don't understand is why we would have to wait for a successful connection three times. If the connection is so flaky that it connects and disconnects multiple times, how would multiple connection attempts make that better? especially when all the "connection attempt" does is check a boolean... (after waiting for some time to give the connection a chance to disconnect).

From the way I read this code, it isn't clear how doing this three times adds stability, it looks like it just adds time (at least 4 seconds, but possibly up to 39 seconds) to each test that uses it.
Can you fix this to use an ID based on the account so that if you have multiple accounts, you can get multiple notifications.
Not only just adding `fail_compilation` tests, but also adding `runnable` tests which @yebblies has mentioned will be better.

```d
void test7603()
{
    int g;
    void foo(int n, ref int r = g) { r = n; }

    int x;
    foo(1, x);
    assert(x == 1);

    foo(2);
    assert(g == 2);
}
```
These files should be TiPropertyObject.h/cpp
Please put propValue->ToObject() into a variable and use Object::Has() to know if each property should be set.
Why did you change ```internal``` to ```external```?
and the same for the other two
methods are named using camelCase
NB: createMergeInputStream supports >2^32 row, but still be < 2^32 rows in memory.
Move this down to line 263.
Indeed. I wonder why it was commented in my ``local.py``, guess I had an old version.
change to `if not parameters:`
Can do, let me whip something up. Could easily be anything -- doesn't really test the validity of the secret, just that it  matches. Kind of a hackey integration test to make sure that Bluepill actually can start services.
I wouldn't call exception message a random string. Why would anyone need to translate the framework and in particular its exceptions that are only useful for the programmers, not to the end users. The programmer must already know English to write program in Ruby and/or webmachine. And in case someone actually translates the framework and the programmer sets I18n.locale somewhere to render translated output to the user, the exception would be translated in possibly some exotic language. The application logs would be incoherent and polluted with random strings in different languages.
Yes, I don't think this method is necessary anymore.
Do you mean Coding Standards?  Please send an example of a CS implementation of this kind of logic.
This was moved here because it is never called directly.  It is a "base" view for others to inherit from.  That's my goal with anything that's called base_something.something.  It should have things in it that are the basis for others.  

Does that makes sense?
Simplify:
valueInStatusColumn != null
@christianv For consistency with other widgets, should we be getting back prepared date strings from Rails, rather than calculating from epoch here?
I did not realise .to_s could be run on nil =) nm
Looks like some weird tabs here...
yeah, good point. Thanks for taking the time to review!
my bad, i was afraid of name clashes. now its called sessions as of 1527c86 - should get rid of name clashes and also work better for tests that need parallel sessions.
i was trying to avoid doing it that way as it means we have to store the locator in two parts, before and after the question number...
Can you add requests and it's dependencies to the requirements file.
You never assign to `_boundingVolume2D` or `_boundingRectangle`.
Any concern about dividing by zero?
Both `setPositions` and `configureExtent` allow undefined input, which has the effect of clearing the polygon.  We can remove this exception and the next one, but have the less than three positions check to be consistent with `setPositions`.
Too many spaces or a tab?
Yeap - have it return an empty object for now.
I think so.  Give it a go and we'll see how it feels.
Nit: `<command:syntaxItem>` is not well aligned
var decs at top and cache children
This should work on mobile web too. If not, it's a bug.
Should it be "or 'down'", rather than "and 'down'"?
Note that MapPointType is a pseudotype. I don't think we should start a habit of naming real classes after pseudotypes, especially since I tend to add "Type" to the end to make it sound less like a real object ;-).
this._title is not defined.
As of Jul 2010, (Read: 1.6 or ealier) iOS actually did support all the colors mentioned.
name of this module is the same as the builtin util. Should consider renaming it to something less confusing.
Kind of a puzzling message. But not a big deal if it's a rare condition.
I think scheduleReconnect is more clear than handleError
Should I change this or is this a hint for future commits?
`Filter` was the point of this pull request, but it uses `Lambda` in its examples and unit tests, so I'd prefer not to pull in `Filter` separately.
If this is meant to be documented, then it needs another asterisk. Are all of these `*TypeOf` templates really meant to be undocumented?
Looks good for functions, but isn't the fact that `@property T defaultInit(inout int = 0);` is a getter instead of being a setter is another compiler bug?
Why would these examples fail? The first example would resort to pass by value so no difference there. The second example would also work because put does not change the range for arrays. What am I missing?
Oh shoot, I didn't realize the default value of a character isn't zero. >_< That might affect some things....
Obviously, this works just fine, but it would probably be better to do `foreach(i; 0 .. 20)`.
no need to default the first argument. Consider changing the name of this method too. The committer isn't "bad", but the group should be ignored.
Provavelmente os zeros a esquerda devem ser removido.
Why not name the variable classname too?
Is there a reason why this shouldn't be an attribute? node['chef-server']['dev']['repos'] or similar.
Oh, nm. Didn't notice the diff.
? Dont you have to do field comments like the others bellow?

Sent from a phone

On 27/03/2012, at 09:24, "Aliaksei Baturytski" <reply@reply.github.com> wrote:

>> +using System.Collections.Generic;
>> +using System.Linq;
>> +using System.Net.Http;
>> +using System.Text;
>> +using System.Threading.Tasks;
>> +using NetHttpMethod = System.Net.Http.HttpMethod;
>> +using NetHttpRequestMessage = System.Net.Http.HttpRequestMessage;
>> +
>> +namespace Microsoft.WindowsAzure.ServiceLayer.Http
>> +{
>> +    /// <summary>
>> +    /// Represents an HTTP request.
>> +    /// </summary>
>> +    public sealed class HttpRequest
>> +    {
>> +        private NetHttpMethod _method;                      // HTTP method verb.
> 
> Yes. It's 4 spaces per tabulation.
> 
> --- 
> Reply to this email directly or view it on GitHub:
> https://github.com/WindowsAzure/azure-sdk-for-net/pull/28/files#r609928
> 
> 
>
the variable name should be httpResponseMessage according to .NET framework design guideline.
Should we keep this file as `@import "responsive"`? Would stop apps from breaking with this update, and thrown in a warning or similar to notify that the file has changed.
what if the suite is already in `self.map`? It'd be more robust to use `setdefault` here, and then setdefault again for the description, more like you do in the add-list method.
Type and visibility should never relate to the naming convention of variables or functions. Hungarian notation habit should be avoided.
`__WORDSIZE` should be determinable from `D_LP64` on all relevant platforms these days.
HI, Sure  you need to put jedis into pom.xml I did not worked with mvn environment before. I compiled the whole package using eclipse IDE.
No, but we can use the same checks that the proxy class does. Something like:

   $groups = $foreman_proxy::dns ? {
      true => ['puppet','bind'],
      false => 'puppet',
    }

and then use $groups in the user definition
How this integration/feature/patch helps test writers/developers
filter.value.isEmpty と書けます。
A change in the encoding - Should not have been committed
I like having an explanation of why `GoogleAuthenticatorRails` needs to be loaded before `ApplicationController` is defined though.  

`'GoogleAuthenticatorRails must be loaded before ApplicationController is defined.'` doesn't really say why that has to happen, just that it does.
It seems a bit strange to me to have the logWrap stuff as part on an example.
No corresponding pushes, so glError() in SwapBuffers() gives a "stack underflow" error. Tested, these can be safely removed.
Not visible here, but the indentation was not good.
def __init__(self, *args, **kwargs):
      for attr in kwargs:
        self.__setattr__(attr, kwargs[attr])
JAJAJAJAJAJAJA xD
I can't find it on upstream version of jquery-validation https://github.com/jzaefferer/jquery-validation/blob/master/jquery.validate.js#L285. where does it come from? I mean how you get regex support in "jquery-validation"?
I think enums are implicitly static.
Shall I add a paragraph about dirty checking being disabled when an interceptor is defined?
it's not necessary to install both mongodb and mongoose. I recommend to use mongodb only
this seems like it will restart apache on every run?
Isn't this already doing the "more correct" thing? https://github.com/MirantisDellCrowbar/barclamp-nova_dashboard/blob/0b16298b31f4cdfe7bbcfd29a08f9517a28502af/chef/cookbooks/nova_dashboard/recipes/server.rb#L160

Or better yet - the apache_site enable action seems to reload apache: https://github.com/MirantisDellCrowbar/barclamp-crowbar/blob/master/chef/cookbooks/apache2/definitions/apache_site.rb#L26
Yes they are.  Reporting 0.7.2.2 and htmlwidgets 1.6.4 are in nexus.  You have to refer to them as:
reporting-api, reporting-omod, htmlwidgets-api, htmlwidgets-omod
That's not right.  The class has a single static utility method for firing a lifecycle event to a single listener.  It also contains the definition for the listener interface.
I'm not sure I  understood that, but maybe I don't need to.

If I create a Ti.UI.Window in app.js, is this path "Titanium.UI.Window," or "app.js," or some other thing entirely?
This parameter should be renamed (to `default`).
You'd probably want to use a leading ``_`` to indicate weak internal use.
still not sure if this is an issue tracker; there might be similarities... but aren't there different types for requirements than issues?
I'm going with your suggestion, plus expected.  For future consideration, a format that asks, "Which of the following REPL sessions is correct?"  This matters in that it's easier to show one line throwing, followed by another line printing.  (Rather than a program terminating.)  (What to show as the thrown exception is another matter.)
There should be a corresponding unittest that would fail with the typo.
for some reason i though i might get it lazy loaded by doing so, but as it was called in `addBackend` during the extension configuration this assumption was wrong ;)
Has this already been URL decoded?  If not, your logic below could get a little messy.  There's an equivalent of http://msdn.microsoft.com/en-us/library/adwtk1fy.aspx tucked away somewhere in the core libraries in the new platform.  You might also need to trim a leading ? depending on where you get the querystring from.
И чем плохо такое форматирование? По мне, использование format гораздо симпатичнее выглядит, чем куча `%`.
In my case that's taken care of by the code that calls Test::WWW::Mechanize::Dancer - and in fact a require would fail for me.

I don't think it's unreasonable for the caller to be responsible for loading the necessary code but now that you mention it, this should be mentioned in the POD.
Ok didn't know this. This caching came from the ``TypeGuesser`` that used it before i reuse the already defined ``getMetadata`` function in the ``Model`` class.
I'm trying to figure out why this entire file has been maked as modified, when each line 'looks' the same?
"enabled as the default image processor" is what I meant with going to admin/config/media/image-toolkit etc. I'll remove that