labofsahil/github-event-dataset-2012 huggingface dataset:
Nope! I don't mind adding here as docs, though the case is covered in the specs. Think so?
Please do not spec the exact error message. We don't enforce them to have the exact same wording, so we don't want to spec the content either then.
Actually, I don't agree with this. From my point of view, the whole spec for Encodings is fundamentally flawed because a lot of the wrappers take a String name for the encoding instead of an actual encoding object.

Would you merge a rewritten Encoding specs where all the wrappers have their native types and just unwrap them? E.g. the `rb_enc_associate_index` would only take FIXNUMs, and `rb_enc_associate` would take an actual Encoding object and just unbox it.
Are outdated, ZF2 agree with PSR-2 and that article must be synced.
Same comment as above -- I think you mean closures, not Traits.
@Mezzle The point is that ProvidesEvents already exists, and does exactly what this one does. One should `use` the other, and that way they can be synonyms/substitutions for each other.
IIRC, getClosingBracket() is primarily for use in determining if the element is standalone (no content -- such as <img>) or content-spanning (e.g., <p>) -- if the former, it outputs "/>", if the latter, ">".  As such, I'm not sure the proposed code makes sense -- I think you'd always use ">".
You don't even need to prepare something if person is not authorized. Later somebody will add another features at top of this function and this new feature will work without authorization. First - authorize, then - do whatever you need. Also AFAIK assignment to a :has_many causes UPDATE.
I like this :)

@tdreyno How do you feel about this?

@stas You can shorten it to `return !!this.get( 'filter' ) === 'completed' ;`

Another question:

Is there a way to make this more generic, so that if I want another filter, a wouldn't have to create a new method on the filtersView?

Pseudo-code: `<a {{action showCompleted href=true}} {{bindAttr class="view.isRoute(completed)"}}>Completed</a>`
Fair point, this was cruft from earlier attempts to test the importer, I've removed it.
@dbu @lsmith77 as already asked on IRC, I need some feedback on this to define what is a field and what is an association :)
I'm going to try FilteredJSONObject; the thing being filtered is the semantic object, not the syntactic JSON.
I like that you refactored common behavior into its own module -- nice work.
supported in group_filter_search
`return list(set(list_))` should work as well, so the `nub` function is pretty useless?

Better nub:

```
def nub(x):
  return lambda x: list(set(x))
```
It's better to use a `booleanNode` when the value is only true or false.

http://api.symfony.com/2.0/Symfony/Component/Config/Definition/Builder/BooleanNodeDefinition.html
This line is an instruction to you, and shouldn't be in the README ;-)
Good point. I'm all for changing it to `slug`.
Oh, nice. I didn't notice that; maybe comment it?
Yes, this will work. You can use whatever you want here.
this one is unncessary,sorry :o
i'll try to give this some more thorough thought soon, probably tomorrow.
Any reason why this particular order was chosen? Wouldn't we always be interested in the last entry?
(last ipv4 > last ipv6 > first 'other' ip?)
this might also be used, no?
this does make sense, do you think however this should configurable? from past experience, people had all kind of dns resolution setup, and it wasnt always trivial to debug why a value is in one foreman vs another.
Any reason to use `\"#{pname}\"` instead of just `'#{pname}'` ?
doh - of course.
no need to change this - it works, of course - but you're aware of the ZU.xpath function, available since, I believe, Zotero 2.1.x that does almost exactly the same as well ZU.xpathText that gets the text, too?
I moved the search feature into a different branch per Kyle's request (and will be sending a separate pull request). I've since changed it to return the raw search results of the rest call so I don't parse through it anymore.
Is that the only "issue" you noticed with this pull? ;)
Why overwrite the table with a Game object, then save a record? There's no table yet!  This is for creating the table.
yeah, was debating to make it a lib so other management commands could use it.  would be easy to lift out wholesale. and yes, it's django settings, so django.conf is the safer way to do it? it's an old habit I have from early django days.
After comma should be space.
What is this guy?
CookieHelper は、capybara-json の追加 lib にしたいねぇ。
This should be changed to id or groupId instead of name.
Should `callback.err` and `callback.data` be wrapped in square brackets as well?
We are passing `limit` in free-form to the CQL query (e.g., string appending instead of as a parameter). We should check to ensure that limit is a number to avoid "SQL injection"-style CQL injection, if that really is possible.
These should become POSTs at some point. No action required yet.
Some comments here would be nice.
my bad, java is too hard for me.
Yes I was. But I will remove the Manifest. I did it for the other methods, not this one.
Get rid of this, its a fairly pointless indirection.
if a user has_many notifications you can do the following

```ruby
def create
  if @notification = current_user.create(prams[:notifivation])
    ...
  else
     ...
  end
end
```

Or if you prefer not to do assignment on the same line as the if use `current_user.build`
The extra parameters should be kept for backwards compatibility. Deprecate, but keep them.
`empty()` returns a boolean value. What exactly are you trying to do with that `> 0`?
Okay, that makes sense ... Should be merged with the other `preg_replace()` then:

    $output = trim(preg_replace(array('/\t+/', '/\r/', '/\n+/'), array(' ', "\n", "\n"), $output));

(improved to catch multiple tabs as well)
Приведенный вами пример является некоректным:
[indexes_array addObject:[NSNumber numberWithInt:i]];
numberWithInt медот это Instanse method, написано здесь:
https://developer.apple.com/library/mac/#documentation/Cocoa/Reference/Foundation/Classes/NSNumber_Class/Reference/Reference.html

а насчет класс-метода string я, конечно же помню, просто выбрал для себя такой вид инициализии всех объектов. да и еще, не всегда нужно зразу инициализировать указатель, прошу взлянуть на строку 19 этого файла где я объявляю указатель и далле в коде вообще не инизиализируется а используется в строке 29.
incAll дублирует изначально известный percentMutate
Как это вы берете остаток от деления на 99 и получаете 99?
Newer versions of CEF adding support for a startup URL, but they just use the name kURL (see http://code.google.com/p/chromiumembedded/source/browse/trunk/cef3/tests/cefclient/client_switches.cpp). We should stick with the same name for consistency.
Where are these weird auto-generated templates coming from?
Also needs Blaster and Description headers.
note, you can do: container.elements.addAll([div1, div2]) here and below.
up to you, the code is fine as is.
A great addition, thanks! Just suggest some cleaner English here.

"using composer's method keeps your code update. Everytime you want to get last updates of all your bundles, just run this command:"

to

"Using composer's method keeps your code updated. Every time you want to get latest updates of all your bundles just run this command:"
Not good enough. This set could be mutated during the copy. It needs to be immutable as an associated object.
add //TODO(thedekel): Figure out the difference between loginLocals/registerLocals
I think this one should be `.SafeDispose()`d once used.
+1 for adding a bool. Returning a hardcoded success message would be a disaster.

An alternative would be to have something like `public void Push(.., Action<PushStatusUpdate> onError)` -- this way we don't have to bother with checking the return value if everything went smoothly.
mne_rt_server
you're right. But otherwise it means doing the copy manually which is more work to do right.

it might work with a trick like:

```
data = self._data
del self._data
epochs = self.copy()
self._data = data
etc.
```
I totally agree, I didn't come up with 'keep_only_data_channels' ^^ :-}
please rename this to jsonJobs or jobsArray rather than jsonObj.
Fair. Default package.json format is with comma before value however.
Indexed correctly in what way?
Changed to Boolean.
Are you sure about this Rafal? shouldn't we leave it null when unset, e.g. in the case where the patient is alive? And thus this method should be getDeathdateEstimated, not isDeathdateEstimated
One of the modules provided by Salt, is the **cmd** module.
It has the **run** method, which accepts  a string as arguments.
The string is the exact command line which will be executed on the minions and contains both the command name and command's arguments.
The result of command's execution will be listed on **master** with the minion's name as prefix.
This is ugly having all this logic in the client. It's much cleaner to wrap the store and put the logic in the wrapper. You can also consider using a dynamic proxy (`java.lang.reflect.Proxy`) in the wrapper to eliminate boilerplate.
```
it 'can change the default iv'
```
GNU Utility
Not necessary here since this is a direct statement of what native handlers are.
you fixed the typo in the other file but there is the same here: ``esily``
`serverName`
I could be reading this wrong, but would this ever execute with the change you made to geometry.rb#19?
Oops and yes!
I've tried but up to know I couldn't get it to work. Will try again :)
newline needed here.
The problem I noticed is that when running rebar ct any folder with a test directory would generate ct logs even without ct tests, having a more specific default for common tests would resolve this, to not change the default at all could be possible but would require a pre-step to check for *SUITEs first.
`currentIndex()`?
Seems like you accidentally hardcoded the output path
It's too bad it's still version 1.8.7 in the CentOS 6 base repo ...
Don't use the same literal twice, put it in a let. (in this case "Nature Enterprise 8/5")
`$s` has no meaning
It is a mistake that we omit the function. Thank you.
What are these imports for?
Sorry, no attention.
I'm saying it should be possible to do nullable=false and _not_ specify a default value. 

Sent from my digital shackles. 

On Jun 9, 2012, at 11:36 AM, Tim Simpson<reply@reply.github.com> wrote:

>> @@ -25,11 +27,12 @@ def upgrade(migrate_engine):
>> 
>>     # add column:
>>     instances = Table('instances', meta, autoload=True)
>> -    volume_size = Column('volume_size', String(36))
>> -    flavor_id = Column('flavor_id', String(36))
>> -
>> -    instances.create_column(flavor_id)
>> -    instances.create_column(volume_size)
>> +    instances.create_column(Column('flavor_id', String(36), nullable=False,
>> +                            server_default="1"))
>> +    instances.create_column(Column('volume_size', Integer(), nullable=True))
>> +    instances.create_column(Column('tenant_id', String(36), nullable=False,
>> +                            server_default="0"))
> 
> Are you saying to switch back to allowing nulls or leave it as is?
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/hub-cap/reddwarf_lite/pull/110/files#r955876
Tässä kannattaa miettiä miten kasaat kaikki sekvenssin permutaatiot sekvenssin ensimmäisen alkion ja loppujen alkioiden permutaatioiden avulla:
~~~clojure
(defn permutations-helper [head tail]
  ...)
(defn permutations [a-set]
  (if (empty? a-set)
    '(())
    (permutations-helper (first a-set) (rest a-set))))
~~~
Yes that's the expected behavior. If the table is empty or pre-filled it will return true, if the table doesn't exist an SQLException will be thrown, the exception will be caught to return false
Good question. Master node initiates MapReduceTask and sends out map messages to all nodes. If map message arrives on node N and node N sends results of map phase to node M before M receives map message then it will not be able to insert result of map phase into tmp cache on node M. The only way to guarantee all nodes having tmp caches ready is to create it on all nodes before map is sent out. Maybe we can somehow create tmp caches on demand but I am afraid we will loose messages in the process....it complicates things quite a bit. Current approach is simpler and easier to understand.
data-role="controlgroup" makes the buttons group together on the page.
Is there any reason why we shouldn't use `cbook.is_numlike` and `cbook.is_scalar`?
Good. Much better.
Can we use the github downloads section instead?
Same here:
(By.ID, 'signatureList')
Fixed. I think it was something I forgot to change when I changed one later on.
Well the thing with const is a bit bad in C#. Const can never be assign (not in the constructor). Then there is readonly (like final in java), which can be set in the constructor but not in subConstructors. Because of the OpenEngSB v2.4 version I have to set these diffrent. (Like createWithId in 3.0 and create in 2.4). This was the nicessest way.
Is this uncompensated? Not calling `staffOnly` for scp requests would allow ajax request without a session?
Should $content accept a string OR array for ``application/x-www-form-urlencoded`` POST requests?
Remove `class="grid"` and replace with: `class="table table-bordered table-striped"`
if you change it in the branch and push it'll modify the pull request.
This looks OK, although it might be a tad more readable if it was done with Vertex-Array eventually. (Especially since it's fixed-points anyhow)
The title should still have a safeUserinput modifier
Can you fix the indentation here and throughout your changes? Also make use of single instead of double quotes and add a space between `){`.
@bp323 The label for should match the `id` of `<input type="file"`
The next version coming up for Sling is `1.1.0`. There won't be a `1.0.7` release from the Sling team, so us using this version shouldn't be a problem. Sling has a lot of bundle releases but I don't recall any minor release for previous versions.
It would. But I would guess that 80/20 rule applies here meaning: in 80% of the cases null should lead to None.
doesn't

```class CMath;```

and then code depending on it work w/o any math include work?
it is not possible to have an SGVector of some non-numeric type, use DynArray or DynamicArray or DynamicObjectArray instead.
just remove the commented parts - it is not legal to use multiclass svms as MKL constraint generators (at least curently)
> If any of the routes middlewares return a Response object, that response will be sent directly.
> The next middlewares and also the controller will be skipped.

The part about RedirectResponse should be removed. Or possibly replaced with a reference to `$app->redirect()`.
Do it like the rest of libuv: https://github.com/joyent/libuv/blob/d07f2466d0a10ab02db588374dd97c2edd54c7fc/src/win/winapi.c#L87. 

You'll probably have to make sure that uv_init() is called, you could add a function uv__once_init for that. See https://github.com/joyent/libuv/blob/master/src/win/core.c#L98.
By significant, I'm assuming you mean more than one nanosecond?

I don't "notice" stuff like that. Seriously, this code isn't a problem.
This will always be true, since an atom is a non-nil value.. you need to 

```clojure
(if @repl-handle ...)
```
Are you sure ``array('error_mapping' =>...pload a smaller file.')`` has the correct type as ``array<string,array|null|...<Closure>|false|string>`` is not a sub-type of ``array``?
need to add this or it will be just the folder
Haha, the you've been won over by the google style guide, I see :)
really really minor issue ..

use `Building.limit(10)` rather than `Building.all(limit:10)` the former will give you a relation, which you can then chain upon, the later will give you an array instead. 

Doesn't really matter here, so it is minor.
Where did the splitting of sql statements go?
Proxy info is only if needed.
But it's much more readable, don't you think?
Remove this cf as it's a internal ticket, not a django-flag specific one
That seems like a reasonable compromise. How would these additional features be defined, though? Would another message containing a PointCloud2 have to be created? In a very specific case I am working on, where I am doing SLAM with ARToolkit markers acting as landmarks, I would like each point to have their "marker id" available for data association, for instance, besides their spatial coordinates.
Can use a default var here:
`BUILD_PATH=${RUBY_BUILD_BUILD_PATH:-${TMP}/ruby-build.${SEED}}`
OK:
```ruby
xs = Array(config.services).map { |s| normalize_service(s) }
```

NOT OK:
```ruby
xs = Array.new(config.services).map { |s| normalize_service(s) }
```
This is the same code I use on [ronin](https://github.com/ronin-ruby/ronin). If you can reduce it, I'd love to copy in the improved version. Tarballs will not contain a `.git` directory, only Git repositories should contain the `.git` directory. Currently, furnace does not run on my laptop env, unless I manually comment out `require 'bundler/setup'`.
Or maybe you should remove this?
I think it is better than {% if field.label == old_password %}, isn't it ?
No, `:` should be in translation string. Not all languages use `:` the same. (French for example have space before it.)
Check [named interpolation](https://docs.djangoproject.com/en/dev/topics/i18n/translation/#using-the-javascript-translation-catalog).
Not to mention that we already do something similar in `FacebookLoginView`.

BTW, `process_channel_subscribe` and `process_channel_unsubscribe` should be kept at the end of the file.
So this can be done with:

```
from lxml import etree
tree = etree.parse('http://findyourfate.com/rss/dailyhoroscope-feed.asp?sign=Aries')
tree.findtext('.//item/description')
```
Yes I noticed them, but wasn't sure if I should delete them
self._unauthorized()
is a function, not a constant, and it returns an instance of HttpUnauthorized()

This is how Tastypie does their tests, so I figured I'd do the same.
Homebrew sets `CMAKE_PREFIX_PATH` to `HOMEBREW_PREFIX` in the environment, is that not enough?
nitpicking, missing comma? Not sure if you want it to be a tuple.
Remove space after `!` and i'll merge it.
You didn't like "attribute_dance"? :cry:
Is inlining the add_avs_details method the right way, or should we just pass the AVSname to the add_avs_details function?
http://pastebin.com/yhj72736
The goal of the placement of the code inside the class is to provide a hind for someone that comes after me, that this is an internal class that hides an implementation detail. Therefor its inside the other class, behind the 'private' keyword and not in a separate file.

I don't care much and not really insist on this, but creating too many files with too many simple classes does not help to read the code.

So if you insist, I will do as you say, but I do not agree.
same as above, these exceptions should use the Log interface to be generated.
usage?
The thing is, if I look at the other RADIUS constants, they are all very specific for RADIUS usage.
In SNMP, we have some more "global" constants...
I agree with you, this does not work well in hierarchies where fields have the same name in different levels.
In fact, the fields filtering feature is not essential for us, and at this point I think that this feature should be more discussed and should probably be submitted as a separated patch.
I propose two different solutions for this:
1. Simplifying the patch by completely removing the fields filtering feature;
2. Re-implement it as top-level field filtering, as proposed by rangadi in previous comments.
Let me know what would be the best solution.
Jeg mener nu han er flodhest
yes i think it's ok (i took it from the big ternet, searching how to validate python identifiers)
mod (`%`) is your friend: `map.gradient = (map.gradient + 1) % 3`. Switching to 0-based gradient values further simplifies things.
make sense will fix that
You could e.g. take TYPE_NUMBER in the block above and make and set 'value' depending on retval.value.type == TYPE_NUMBER.
Remove, see notes below.
Not sure about this, is params['attributes'] defaulting to `{}` ? If not then it may get passed as nil and the method in the readme is misleading.
I was wondering if I needed a fit() function that returned a results class instead of just having the fitted attributes as part of the model class.  However, l'd have a class with only a fit function and nothing else, which didn't seem right.  But, it might be worth it to stick with statsmodels conventions.
This is a great refactoring of the Hangul stuff, nice work.
Please align properly.
If God finds you worthy, the link will work without an &lt;a&gt; tag.
Yes, I think so too...
ooh, dinbats in comments!  :+1:


On Thu, Nov 15, 2012 at 10:07 AM, Jason Bau <notifications@github.com>wrote:

> In chef/cookbooks/class2go-apache-restart/recipes/default.rb:
>
> > @@ -1,3 +1,7 @@
> >  service "apache2" do
> >      action :restart
> >  end
> > +
> > +service "shibd" do
> > +    action :restart
>
> oooh fancy github stuff if you enter ":"
> [image: :+1:][image: :-1:][image: :100:][image: :109:][image: :1234:]
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Stanford-Online/class2go/pull/1308/files#r2142143>.
>
>
remove above comment block (now in LoadBalServer)
Do you really want to truncate the application names to 20 characters?
[Globalize3 : Scoping objects by those with translations](https://github.com/svenfuchs/globalize3#scoping-objects-by-those-with-translations)
Para manter os parâmetros recebidos via GET, podemos fazer assim:

    import urllib
    urllib.urlencode(request.GET)
Shouldn't there by a try/catch here?
ah, right. This should be removed then.
This is important for people who whish to implement emitters - while the compact fluent interface and compact return syntax require no changes (because they're already being transformed to their expanded counterparts by the grammar), **this here does**. An assignment parameter looks like this:

```groovy
// function($this.field)
$param= [ assign : 'field' ];

// function($this.field= null)
$param= [ assign : 'field', default : new NullNode() ];
```
Yeah that's a good point it.  People hacking around something would be pretty frustrated with the extra hurdle.
ImmutableSet.copyOf(
@dzuelke No need to create subclasses. The static property is public.
What is the deal with these lines, Hans?  Please add a comment at line 103 explaining why the following code is commented out.  It's esoteric and should be explicit.
Revert this file, it already has a check for this.  Also: It's a lot more lethal to go down our disposals.
Yeah for sure let's fix this
Bonne pratique :).
Whenever reasonable, we want the string keys to basically be the string literal itself.  That way, when looking at the code, it's easy to see what the code is actually displaying without having to look up the string resource.  So this should be ConversationFragment_delete_message_confirmation
I'm thinking prod, and testext will actually point to a live memcached, though we can play with those later
モックをテストしているわけではないのですよね。
「クエリー」、「クエリ」は他の箇所とあわせて統一したほうがよいかと
there is no --with-internal-ffmpeg. If you set --with-external-libraries then that implies external-ffmpeg!
Why aren’t you using ``make_regression`` here?
I would rather say "number of samples". Amount is only for uncountable things, right?
I agree that memory consumption here is not such a huge issue but IMHO the code complexity isn't affected much by the additional ``if sample_weight is not None`` checks.
Unless there is a good motivation for adding it to kernel PCA, we can just rename to cosine_similarity. Or we can have cosine_similarity with cosine_kernel as an alias. cosine_similarity is indeed a much more common name.
On Thu, Oct 4, 2012 at 1:15 AM, Mathieu Blondel <notifications@github.com>wrote:

> In sklearn/rbm.py:
>
> > +
> > +        np.random.shuffle(inds)
>
> You could do del self.random_state at the end of fit then.
>

What if fit is called twice? Wouldn't having deleted the self.random_state
cause some problems.




 —
> Reply to this email directly or view it on GitHub<https://github.com/scikit-learn/scikit-learn/pull/1200/files#r1758454>.
>
>
Hm I thought  you could eliminate this one, too. But I guess it is not so obvious. I think you'd need my favourite non-existing numpy function ``bincount2d`` (analog to ``histogram2d``). One of these days I'm gonna write it ;)
Same remark for `n`.
Needs a TODO
"for those you can" which would mean you can use forward declaration for Container and AbsoluteLayoutProperties, which are only used here for pointers. Color needs the full definition so of course you can't use the forward declaration.
getStatus is not an expensive call so maybe we can reduce the wait time from 10s to something smaller, like 1 ? Wdyt?
Das mit dem Lookup habe ich aus Performancegründen gemacht. Die Performance ist massiv besser, wenn man nicht für jedes Dokument die Bewertung separat abfragen muss. Siehe auch: https://github.com/gwrtheyrn/studentenportal2/blob/master/apps/front/views.py#L297
maybe expandGraphText();
On a second note, Gavin was thinking if row variables shouldn't also be promoted to 64bit, but I'm not sure temp tables will ever be that big, especially COUNT tables, where COUNT would have to be a 64bit quantity as well. (is it?)
If we don't have enough space, maybe we should just bail out or have a dynamic buffer instead of trying to unlink() the wrong file name.
I'm not sure that a file name can't be larger than PATH_MAX, which is a fixed value only on Windows.
This was the simplest way to deal with reset! raising WebkitInvalidResponseError after a failed #visit.

When PageLoadingCommand gets an error (as happens in this pull with the new rspec example), the driver can raise WebkitInvalidResponseError twice. From the initial #visit, PageLoadingCommand::pendingLoadFinished returns an error status. The next command (in this case, Reset) will also return an error status because Connection::m_pageSuccess will be false.

I think that changing the driver so that Reset returns success after a failed Visit (and removing "rescue nil") would be an improvement. How does that sound?
I'm confused: why does this take a block?
Sure, will get ahold of you offline.
``isOauthExists`` ? why 2 verbs in the name ?
typo @ "thoso". Also, "interested to use" -> "interested in using"
I assume this is 10 seconds. Should this value be configurable in some way? or is that overkill?
If you put the port as an arg to ```__init__```, rather than ```start``` and keep it as a private member, you can specify the port when using the object as a context manager. e.g.
```python
with FusekiServer(port=123) as server:
    #do stuff
    server.save_cache()
```
This whole block of code needs comments to describe what's happening.
What version of facter are using for that? I *think* that was fixed in 1.6.6.
Huh? connected = false
Is this a todo that can be accomplished before merging? Merging code with todos inside is no bueno :)
Trying to find a definitive answer, not MSDN but:

'The Sysvol on domain controllers is used to deliver Group Policy settings and logon scripts to clients at logon. Since Sysvol is used for authentication of users and computers, it must be up to date on all domain controllers. When any information is changed under the Sysvol on one domain controller, it triggers replication of the Sysvol to all other domain controllers. '

http://www.windowsnetworking.com/articles_tutorials/understanding-group-policy-replication.html

So best to add a single DC, or change enum_dcs to return a single DC.
Testing for today's commit - I decided to add a singleton dimension during input parsing in the gray-level case, which will permit the more elegant 2nd recommend form.
btw what is 'index1' and 'index2' ?
Double default?
I think we put spaces around the colons.
As I mentioned elsewhere, we may be able to remove this by traversing the quadtree front-to-back.
Will users do this?  Or call `PolylineCollection.add`?
Should we break this long line ?
hmm, what's the reason for using mvel here? better stick to JSP to avoid an external dependency just for this error page.
Why was the previous description changed?  We should call out that appendrow can take a list of rows.
Описание полей хорошо бы выстроить в одну колонку. Ты же не Маяковский ;-)
```
 * @param {String}      [name="Новое событие"]            Имя события
 * @param {String}      [description="Описание события"]  Описание события
 * @param {Object}      Parent                            Предшествующие связанные события
```

Так ведь на много лучше
`Calendar.prototype.constructor === Event`?
They would have been needed (well `ev_ref()`/`ev_unref()`) back in the `eio_custom()` days, but `uv_queue_work()` references the event loop for us so you don't have to do it manually.
there's a hard time trying to both test forward streams, and develop on backwards streams. If i had to allocate my time, i'd prefer it's on the forward streams, to catch bugs.
>version added: 1.4

:open_mouth: Старею.
Впрочем, давай пусть будет хидден-фича. Мало ли на стейдже нужно будет отладить.
The guess value really didn't need to be an array:
```
            <div class="guess"> [ angle ] </div>
```
I added a new commit to this push that sets the guess to a single value rather than an array.
```
            <div class="guess"> angle </div>
```
Places that use the guess were modified as well.

@spicyj Thanks for the tip.
Random access isn't actually relevant to this. It's the fact that `hasLength!R` is `true`, so the comment is misleading.
OK fine with the current convention on dummy ranges.
Actually, that's not enough. If the subranges are themselves ranges of ranges, then *their* respective subranges may be consumed if the user iterates over the .save'd range.
This is the line that directs / to the products index page. to rake routes in a terminal to see all valid routes.
wow, this change is funny
suggestion: maybe create a typedef for the function type, like:

    typedef void Action();

(or maybe we eventually get something like that in corelib...).
alternatively, maybe prefix rename the function "_afterN", it seems like its not really intended to be used outside this library
single quotes for keys :)
I agree it'd be nice to allow multiple customizations, I'm not sure how I would use composition though - do you know of an example I can look through?
I meant to tag this import of GoogleAnalyticsTracker as the one we don't need.
I get your point but...most "pretty urls" use either dashes or underscores to separate words in a URL. 

I think it enforces a good/common practice...what do you think?
start_date.isoformat() ? or does the above format better match what candlepin is actually returning? 

They do seem to be different, though I guess they could both be valid iso8601 format
why don't we check for interfaces here .. seems easy enough to do properly right away?
Actually, just an else, right?
I think it's the way folks do it in phobos.

There are 67 such comments in Phobos and 0 in druntime.

Although I have no idea why such syntax.. I agree that it doesn't look cool, it doesn't even get highlighted in IDEs etc...  
//FIXME: Bugzilla 8675 - looks better, get highlighted in IDEs and is listed in TODO summaries.

Just my 2cents :-)
Why is this hardcoded to last month?  It should be "new Date()" (and formatted as such).
storedProcExecutorBeanName
I tried :-) In the olden days there was a property "buildDirName" which does not work anymore. "buildDir" however resolves to the absolute name which does not work. I think the problem is that the sonar plugin is not well implemented in regards to Gradle. Basically, props["sonar.jacoco.reportPath"] is applied for all sub-modules but configured under the root module.

Thus, I need the pure name "build" not the absolute path. And since I did not see some build-in function that does the conversion I went the path of least resistance and just hard-coded "build".
are you sure this isnt required? do you know exactly what this flag is used for?
Any reason to use ECB instead of CBC?
Also above it, bin/reddwarf-manage, dont we have a reddwarf_manage or some method like that?
i think for /my/path this would match /my/pathSomething which is wrong. it must be == $srcAbsPath or like $srcAbsPath . '/'.

the other thing i wonder: can't you do the replacement inside sql instead of loading and then storing the node? this will be extremly slow for a large set of nodes.

i think you should use query builder here rather than raw sql, to make it work on a maximum of platforms.

can you try to do this please? if we don't figure out how to do it, at least fix the where criteria, and we can ask the doctrine community how to do the substring operation.
http://stackoverflow.com/questions/8619421/correct-syntax-for-doctrine2s-query-builder-substring-helper-method
Please don't leave commented-out code in the commit without adding a comment that explains why it was just commented out and not removed completely. Also, there is now a semicolon missing at the end of the statement (because it is now commented out :-).
Ok, let's leave it as is and get back if this becomes an issue.
що тут робить бекап конфігу?
Must i inject RequestContext in the controller ? Or i can find it somewhere else ?
????

What's the point ?
Please just delete things instead of commenting them out; we can find out what used to be there from version control if we need to. (See http://nedbatchelder.com/text/deleting-code.html for discussion.)
django-admin.py startproject

The developer can remove or add whatever apps they need to.
You might want to change your description implementation to fit more inline with what Apple provides.  For example:

- (NSString*)description {
    return [NSString stringWithFormat:@"<SFOAuthInfo: %p, authType=%@>", self, [self authTypeDescription]];
}

Notice that it prints the class name, the pointer value of self, and the whole thing is wrapped in brackets in the event it's NSLog'd along with other stuff.
Can this be merged with the code in `DeclarationExp::semantic` ?
this desynced too.  it was the attachment = { mimetype: 'text/plain' } bit.  So do  attachment = { name, type, etc etc} as done in imapchew.js.
huh? look a little closer.
This is not supported in Android.
The file documentation.js is nowhere to be found .. is it still needed or just missing ?
Nothing noteworthy in this pull request, except this trailing comma which should not be there.
I you want, please go ahead. Thanks for you help!
jake lint did not catch it.
But on the other hand it gives you a possibility to have different SPIs per test method, right? So you define the context first by creating SPI file and then stubbed implementation returns for the given tests all SPIs you created.
Let me double check
So in the webworks whitelisting scheme, the user would need an access element to do these things. As per file, its covered in the cases below.
queryForObject expect to find one an only one result else throws a special exception.
why do you eliminate the possibility to change the template? i think it could also be something that people want to configure.
and the docblock is now incorrect.