labofsahil/github-event-dataset-2012 huggingface dataset:
Not necessary, line 51 does equivalent
No horizontal rules, please
But doesnt look good either
IMO, the value should not be limited to string.

An example:

``` php
<?php

$table->setOption('temporary', true);
```

Allowing to generate:

``` sql
CREATE TEMPORARY TABLE (
    // ...
)
```
I mean you can just do "it_is" instead of explicitly returning.
On May 12, 2012 3:08 PM, "David Kullmann" <
reply@reply.github.com>
wrote:

> >    message = "What is #{something}?\n"
> >    message += "[#{default}] > " unless default.nil?
> >    print message
> >
> >    it_is = STDIN.gets.strip
> >    it_is = default if it_is.empty?
> > +  return it_is
>
> incase you can't read that last comment, I'm saying that "I know that" and
> that:
>
>    it_is = default if it_is.empty?
>
> Will always return 'nil':
>
>    1.9.2p320 :008 > it_is.empty?
>     => false
>    1.9.2p320 :009 > default if it_is.empty?
>     => nil
>    1.9.2p320 :010 > puts if it_is.empty?
>     => nil
>
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/josegonzalez/cimino/pull/1/files#r812645
>
Only one var statement.
Size of a same object can't change
I guess I'm curious because `login_user` didn't exist before this, and isn't referenced anywhere else in the code.
We shouldn't return a plain array here. It should be wrapped in a 'results' key. (see libraries, etc.)
Indexing tasks VIA RabbitMQ are implemented, as well as node-level events to listen and fire those events.
Yeah,  `comm` is probably better. We definitely can't use `c`.
This should move to line 1, so we have the imports from other modules first.
By the way, is it possible to get other exceptions?
Nice! even better:


    for {
      resource <- candidates
      _ = pm.worked(1)  // interpose a side-effect
     ...
This isn't what I meant - this class needs to be called "FieldGenerator", and "MiAttemptFieldGenerator" needs to derive from it, so these common methods are only in one place
Is this being set when the document changes?
Breaking the style guide rules - exclamation marks must have spaces around them.
Save a few bytes:

    isset($route[1]) OR $route[] = $class;
    isset($route[2]) OR $route[] = $method;
По идее 100.0 — уже будет float, так что можно или f убрать, или .0
percent > 100 or percent >= 101
Вы сделали массив ATGC глобальным. В тоже время вы инициируете его каждый раз при создании нового объекта. Все преимущество "глобальности" исчезает. Наверное было бы лучше сделать таким образом:
	if(!ATGC) ATGC = @[@"A", @"T", @"G", @"C"];
there is a typo, which travis caught - `$this->ressource` should be `$this->resource`
Denne trengs fordi Buster og asset-pipeline ikke kjenner til hverandre? Hmm.
FYI you could have used np.pi
I had in mind that the one compares memory addresses / object identities and the other value-identities. Thus I always try to use 'is' for the one purpose == for the other. In this case it makes no difference and also it's fine with me.
Agreed, I haven't polished everything because I am not sure if you even want to replace the dictionary representation for labels
Should render(:partial => '...', :collection => @item.comments)
not ``a`` driver. In 2.0 and 2.1, ANSICON is the only supported driver, because of the way the detection works. The support of ConEmuANSI has been added in 2.2
Thanks! - fixed in sha: fe6632bdae368bc1b181e0ff66f164d7ebce6364, though some of the new hashes changed to ids, which is kind of a bummer :/
You should put an empty line before this line
I think it's Ok to be conservative here, this code just need to be clear, not fast.
I don't think this line is necessary.  The `with_connection` call should be enough to check out a new connection.
The following two lines should likely be inside an ensure block. So if a test fail its state doesn't leak.
should also make getRestPrams() to be public.
`HEADERS` и вон ту хрень с `QT4_WRAP_CPP` вообще в новых плагинах можно убирать — в cmake 2.8.6 и позже есть поддержка automoc. Ну и, соответственно, `MOC_SRCS` тогда уберется из списка исходников для `ADD_LIBRARY`.
Тебе тут в хедере точно оно нужно?
Again, lots of hardcoded values that prevent the test from being run a second time.  Need to produce unique values (see uniqueify).
If the classes casino1 and casino2 both included a module with the onearmbandit method, both would be using the exact same implementation, the one on the module they both included.
Non, on attent une 403 ici
learn how to quickly produce **games** using standard gaming
Need to keep these in there, so we don't break older projects. Or is this a hard deprecation?
This doesn't like up with the other parameters.
Hey, the first line should be `A`.
Should this just be 'printed', rather than 'preprinted'?
That's just a typo -- will fix to match the other.
If you run `python setup.py develop` as directed then you will have gaiatest available on the command line
spacing ?
Should you show a warning for this on the install/upgrade page?
Thats extremely unreadable.
What is this?  Probably should explain or break out the start-idx, end-idx of the TimeSeries creation -- maybe into the let binding.  This is strange.
If I'm understanding this line correctly, you're calling the element_start method from the superclass, which is a nop that always returns False. This seems like it would hinder performance for no reason. Or am I missing something?
@apex13 Can we update the modified lines to align with our style guide?
Does Russian even have colons?
Hmm.. neither MoveOperation nor SakaiDocPagePublishOperation should have any of the \@Reference annotations that reference cleaners -- those have been moved into the SparsePostOperationService.. not sure why that error would be happening.
Why was this changed? It's broken the breadcrumbs when you're viewing member grid in a group.
oops. probably didnt mean to delete this one ;). I re-added and also added a simple spec since I neglected to do that when I added it first time around ;(..
useless else after a return
No worries! :) This is much cleaner.
I would throw this as an UnexpectedValueException like this:

    throw new UnexpectedValueException(sprint('%s::getFieldAttribute `xml` is not an instance of JXMLElement', get_class($this)));

That way it's easier for the developer to home in on where the error is.
Kan se sånn ut! Will fix.
J'avoue, j'ai pas encore testé le bilboplanet avec php 5.4 ... mais si c'est la seule erreur que tu as pour supporter le php 5.4 c'est plutôt une bonne nouvelle, ça veut dire que dans l'ensemble, le reste est supporté.
No need to change that for development. Please pull from main repository changes I made which configure e-mails to be output to the console instead of sending.
You haven't added `event` here.
Indentation is a bit off?
Why adding `staticfiles` here?
Don't create HTML fragments manually, use jQuery. Check how we are doing it for posts.
```
$('<div/>').attr('id', 'advanced-map-container')
```
Oh, no, work on `obj` directly. There could be difference between `obj` and `bundle.obj` at this stage.
For constant stings, use `'`.
Well, if I'm correct, we have a lot of station models in our database with the same id, but different timestamps. So if we use the query you mentioned above, we would get all stations nearby, but not just one object for each one of them. We would get objects with specific station_id, but with different timestamps. So I first wanted to find all stations nearby, and then retrieve the most updated info about them with ordering by timestamp. I know it's time consuming, I just didn't think of anything better.
You should always do such things on server side. You never know if client has done it properly.

This is also why you wait for server to return/push the post even for the author. And should not just try to be smart and reconstruct the post in JavaScript. (Or if you do, you must know the consequences.)
So all those are available in `foursquare_profile_data`?
Put one empty line between initial one-line description and more precise explanation later.
You can use `user/%s`.
I renamed it in my code and then checked if everything works fine, but it returns ValidationError, when doing a query mentioned above in a task. If I don't have it named "id", it works fine. Is something wrong with defining a field with "id = mongoengine.IntField(required=True)" ?
Changes would break Android+iOS, wouldn't it?

The indentation changes are also pretty annoying :/
Is sleeping the right thing to do here or prompting Y/N to continue?
No need to print the list of gateways. You could, however, print the total number of gateways to be tested.
My fault, this will be corrected -.-
45sek after enrage, you reach this point, and get stuck in endless loop
@dlsniper If you retrieve the service in the constructor, it causes the same scope issue than retrieving it outside the constructor and passing it. the logic runs at the same time: during the service creation. The only difference is that the container cannot detect the wrong logic. You should read the [doc about scopes](http://symfony.com/doc/current/cookbook/service_container/scopes.html). You should retrieve the asset helper only when needing it, and not storing it in a property.
I was thinking then as `Object` only... After all it is an object pushed around...
This should be "GitHub" and not "github". See @kneath's [tweet](https://twitter.com/kneath/status/275841609151700992).
We always give documentation in the form of `1/1/xx` and then explain where `xx` is ...

You should check other examples in the guide and align to that practice.
That's fine. Then I'd recommend a different variable name.
The connector implements the EventSupport-Mixin.
Remote-connectors store mixins in their config.
Not sure what you mean. The zoom area would "snap" to the visible events when filtering, yes.
Extra space before `)`
Sure. One suggestion is to make test_corr a function _test_corr where the arguments are not optional. Then test_corr the method can call this function with sensible defaults. And ci_corr can call _test_corr giving default values. This goes for all the other methods that follow the same pattern.
type: float
Move this to settings.
My suggestion is you wait until you review a bit more before doing another pull request.
Just a v.v. minor typo (which I see is inherited!)
Sorry i must have got confused while merging in, i remember this line and figured i must have broken it at some point and just guesed that it should have been xt.
This is from that issue. I'll look into this in the engine code.
I don't know. But the test has changed itself, so not a change caused by formatting alone
Add a `public static BrooklynCommand parse(String...argv) throws ParseException` method to make this testable
But isn't that *okay*? We're testing if we can safely call `a.isEqual`, nothing more. `"function"` means we can safely call it -- even in those browsers with callable RegExps. Sure, they had some bugs with `Function.prototype.call` or `Function.prototype.apply` being applied to them, but for simple invocation, this test is fine. Agreed that a JSPerf would still help in the decision-making, but I don't see the `typeof` test being slower than... anything.
yes, since dialog box is outside of view.
Since `forSetting` is being removed, this will go too.
Even though I know it will pass, does it make sense to test a `String` instance on 1.8.7 .. since that is a bit of a weird case? I'm thinking regression prevention if the code is ever refactored or reworked a bit. I'm not *sure* it's a good idea, but I did have the thought.
I've removed the migration and put the enforcing in the save method:
https://github.com/tangentlabs/django-oscar/commit/166d7be192798391074801ff413926399bee96d3
see new comments and variable names
We don't need this too.
Cause I fixed the loops so they wouldn't happen :)
also never used
iaex надо передавать в конструктор в качестве cause
Ты тераешь саму причину - Exception
It says size_t::

schweizer_messer\sm_database\include\sm\database\implementation/SerializedMap.hpp(133): warning C4267: 'argument' : conversion from 'size_t' to 'int', possible loss of data
You are describing the implementation, which is not so interesting to users. You should describe the *behaviour*, like is done for the first two (delete only snapshots, delete snapshots and blob); what does null delete? *Why* would someone want to use this?
It should be both. It should work out-of-the box (and for tests) and exist as documentation.
Why is this calling a one-line `system` call?
ですよねー。
61c27c26634bf113e6f4459a8630408c7dbb7b56
こんな感じでしょうか。メソッド名長いですねい。。
Because 'formType' is only one of the types of classes that it will be handling, it was mentioned changing it to 'inputClass' previously to make it more generic
what about using a different name to be more readable ? and what about using ``&&`` instead of nesting 3 ``if`` without ``else`` ?
Look at line [125](https://github.com/nodejitsu/node-http-proxy/pull/254/files#L2R125).
change the name from @links to avoid confusion with the @links variable on the next line
No problem - how's this?
if you think ahead to enet_path you'll have the big picture which will help to get a better design.

For alpha just try and give some feed back
Really? Ok then.
Or is this a test for the backward compat code?
Could you please find a better name here. :)
Le 24/10/2012 23:38, serge-sans-paille a écrit :
>
> In pythran/pythran.h:
>
> > +    };
> > +
> > +template<class K,class V>
> > +    struct assignable<pythonic::core::dict<K,V> >{
> > +        typedef pythonic::core::dict<typename assignable<K>::type,typename assignable<V>::type > type;
> > +    };
> > +
> > +template<class T>
> > +    struct assignable<pythonic::core::list<T> >{
> > +        typedef pythonic::core::list<typename assignable<T>::type > type;
> > +    };
> > +
> > +template<class T, class U>
> > +    struct assignable<__gmp_expr<T,U> >{
> > +        typedef mpz_class type;
> > +    };
> >
>
> ce dernier template (tout comme le #include gmp...) devraient être 
> dans un fichier pythran-gpl.h qui serait inclus dans interface.py (je 
> crois)
>
> —
> Reply to this email directly or view it on GitHub 
> <https://github.com/serge-sans-paille/pythran/pull/21/files#r1934679>.
>
On en parlera peut être demain.
J'ai réussis a mettre le flag no_gpl et il fonctionne!!

Par contre, il y a une partie que je trouve pas très jolie. En changeant 
mon flag no_gpl, soit je change pytytpe_to_ctype_table mais c'est dans 
table donc j'ai pas bien envie de le modifier depuis script/pythran. 
Soit, je fait la modification "en dur" dans la fonction pytype_to_ctype 
(qui est dans interface elle).
Seul problème c'est que j'ai besoin de cette fonction ensuite dans 
typing. et import pytype_to_ctype from interface ne fonctionne pas. Je 
suis obligé de faire import interface et de rajouter un interface. 
devant mes appelles. J'ai d'ailleurs le même problèmes dans backend.py 
ou l'import ne marche pas.
As tu une idée?

Bonne et longue nuit :-)
Tell us your thoughts here, It looks like the TODO comment might be dangling. Perhaps we should try GPU devices explicitly then fall back to ALL?
From the old code I understand realms were not locked.
if bSucceeded is false, that should be logged too
Probably shoud disabledBackgroundColor_ instead of backgroundColor_
After nextEventId reaches 2147483647 the next increment will cause an overflow and nextEventId will be -2147483648
I believe you're looking for `toValue` from the `Text.Blaze` module. There is an instance of this for numerics, so we should be able to do `circle ! cx (toValue $ fst $ position)`.
This is not valid for the PDO storage engine. This should issue a delete statement
ah got it. Yes I think you' re right.
Should you be doing something with status ?
You're supposed to merge when the patch is reviewed and accepted. I was just raising the flag so you didn't pull it before it was safe to do so.
Same code occurs 3 times, so I would be tempted to make it into an inline function.
e.g., getActualRows()
Being fussy these could probably be const functions, assuming they can't logically change once the IReferencedFile is created.
Moved to a local var in CStableQuickSorter
what a dummy mistakes...  sorry.
My bad! I got lost in all the horizontal scrolling which is hard if you don't have a trackpad. (currently on external montor plus usb mouse and keyboard)
this looks like it just readds parted. Am I missing something?
For "real" hogan they're just fragments of text (or html if you're doing html stuff...) The part where you load the file isn't handled by hogan.

As for the partials part, I didn't take into account that they were to be relative to a certain path or something like that, just considered that people would have to input a correct path. Do you have any suggestion of modification? Something similar to the whisker's implementation is doable.
Is deze wijziging wel juist ?
Why not replace the whole body by `return getInteger(0);`?

The same counts for all the other get* methods, except maybe for getString()
I don't understand this comment, but if my guess of your comment is correct:

`typeof(value) === 'object' // is true for null`

but

`value && typeof(value) === 'object'  // is true only for objects`
It may be worth it to specify that `contentScriptFile` (and `contentScript`) accept Arrays.
In this whole page, there is no mention of that, whereas you expose in this paragraph that we can load multiple scripts,
but without "real example" (i.e. real addons should not use `contentScript`, but only an array for `contentScriptFile` instead.)
Please use brackets around the condition to make the line better readable.
In the original code the $array was converted by determining the classname of the first item in the array and using that for the other items (thus assuming the array contains the same objects). Getting the first item out of the array used the key() function.

After my changes the $array variable can be an array or an object implementing the Traversable interface. I believe an object implementing the Traversable interface doesn't have to support the key() function. So I used foreach to determine the classname of the first item instead.
Refactored using ```strpos()```
This goes a lot easier with destructuring:

~~~clojure
(defn contains-point? [[[bl-x bl-y] [tr-x tr-y]] [x y]]
  (and (<= bl-x x tr-x)
        (<= bl-y y tr-y)))
~~~
I'd collapse these two lines down to just ``return self.obj_delete_list(request, **kwargs)``, since you're not doing anything complex nor doing anything with the intermediate variable.
Right, it makes sense to do the brackets before the parentheses.  I also separated the three commands so it looks less like gibberish to a beginner.
I'm confused.  I thought you created type above - `this.type = undefined`.  Does this just make it read-only?  Or does it recreate it?
Should be "...that extends infinitely..."
Should this be a `Cartesian2`?
As you suggest, this is only marginally useful now.  In general, we need to think bigger about the robustness of creating a context.  For example, if any of our shaders fail to compile, we should be able to detect that and bail.  Likewise, we should bail if the frame rate is really low.  I'm thinking more like the Google Maps WebGL test.

This pull request is sort of a first step, but all it really does is wrap `typeof window.WebGLRenderingContext !== 'undefined'`, which is not the whole story of supporting WebGL.

Let's discuss on Monday.
Remove unecessary xml element for role ? It's empty anyways.
Why commenting and removing?
space after opening { and before closing } Also, this will create a border at the bottom of the last row, but not a border at the top of the first row. This imbalance needs to be fixed.
Same as above.  The requested change is to handle developer error.
There's an awful lot of widths on this line.
For the error condition where the user sets the maxZoomScale less than the minZoomScale and then switches it back
var sv = Ti.UI.createScrollView(); //minZoomScale = zoomScale = maxZoomScale = 1.0
sv.maxZoomScale = 0.5; //This will force the zoomScale property to 0.5 and minZoomScale is ignored (IOS behavior)
sv.maxZoomScale = 2.0;//Unless the above check is performed zoomScale will remain at 0.5 which is invalid since minZoomScale is 1.0
Yeah, I will feel better crossing that bridge when we get closer to that and have some more concrete user stories / full picture.  Personally I would rather avoid over designing for future behavior we don't have a full picture of yet.  We might address a change here but I think we should leave this be for the time being since it's simple, obvious, extremely low overhead and works.  My 2 cents.
args should not be null. They might be empty. In that case, you should set dest = '/'. I think this whole section could be tweaked a bit.
I agree with you in the general case. However, I think I disagree in this specific case for two reasons:

1. Seeing dart:isolate really scares me. I don't understand what's going on. I tried to read the source for the timer stuff, and I couldn't figure out how it actually works.

2. The layer of indirection provided by dart:isolate's timer class doesn't provide any benefit. Maybe if I were trying to write code that ran both client and server, I could see the benefit, but in this case, I think it's better to "keep it simple stupid".

If your really disagree with me, we can get @vsmenon to weigh in :)
Let's just use the filename without extension for now.
@andralex: `std.typetuple` primitives should not take functions as predicates. To be able to say this with confidence, I'll just play the »I've done a lot of related work« card: I've used `Filter', `staticMap`, `allSatisfy` and friends quite a lot, and yet have to encouter a situation where I would want to use anything but a template as a predicate/map. Also, we've had `staticMap` for ages now, and I don't think there has been a single bug report asking for function support so far.

Templates are the natural fit for such algorithms, since they can operate on both values and types, and are always »evaluatable« at compile time, both of which are requirements for the use with compile-time (»type«) tuples. The only case where using a function makes sense is when a) the function is CTFEable and b) the tuple actually only contains _compile-time_ expressions.

Granted, it's not at all impossible to come up with situations where the latter case occurs in practice: for example when processing the result of `__traits(allMembers, …)`, which is a tuple of compile time strings. The question is just: Is it worth to complicate both the implementation and the API[1] of every single of the related primitives, to make the code for an infrequent use case a little bit shorter? I don't think so.

As far as `Lambda` is concerned, I'm a little bit torn. I'm certainly not overly happy with the name: It doesn't really express the semantics, while potentially being a source of confusion w.r.t. lambda functions, as you mentioned. Maybe `ToTemplate` would be a better choice? Besides that, I'm also not sure if it's used often enough in client code to warrant inclusion in Phobos – even more so because it's trivial to write (or just manually wrap a function into a template) if needed.

[1] With templates as predicates/maps, it is immediately obvious for readers of the client code that whatever is going on happens at compile time. With functions, not so much.
This function might be a better match for std.process than std.path.
Environment variables are useful for way more than just path manipulation,
and it's already tied into std.process in a way by using `getenv`. I'd like to hear
what everyone else thinks.
@Xhamolk, already corrected. I guess github doesn't give email notifications for edits?
While this is under heavy development - I would say use the git repo and the master branch
`@param uo` -> `@param object`
No, by far not all of it. I forgot to publish another change, so this one is out of context. I'll publish the complete change today.
shouldn't phing do it in its composer.json instead ?
I moved it into the serialization. 

I was on the assumption, that you can not log into a user, that is not persisted, because the `UserProvider` would not be able to load it. But yeah, the serialization is not bound to that case completely.
This error message is true, but it should be phrased in terms of what the user is expected to do. Something like:

> Message cannot be deleted because LockLocation is empty. Only peek-locked messages can be deleted
tests are added.
instead of using parseDate here, should we do the date parsing in PoolWrapper?
May write this as:
```js
	testMetaDoc: /^\s*<\?xml(?:.|\r|\n)*?xmlns(?::.+?)?=('|")(?:http:\/\/www\.metalinker\.org\/|urn:ietf:params:xml:ns:metalink)\1/im,

	getMIMETypeFromContent: function(req, data, length) {
        data = String.fromCharCode.apply(null, data);
		if (this.testMetaDoc.test(data)) {
			if (req instanceof Ci.nsIHttpChannel) {
				req.setResponseHeader("Content-Disposition", "", false);
			}
			return "application/metalink4+xml";
		}
		return "";
	},
```
* ローカル変数はキャメルケースで記述しましょう
The naming looks weird. The visitor is not nullable. It serializes null values
might as well put the quotes around $password here too
this не нужен, амперсанд не там. на верхнем и нижних методах тоже. Скобки return (); не нужны. this тоже
Cache manager created but not stopped. Use the TestingUtil.withCacheManager() method to avoid this.
@nicolaasmatthijs So maybe not in the source file but in another location? It's not easy to find where we actually got this script from.
If `STORE_PATH` is variable (ie not a constant) we should make it lower case like any other variable
Using a regexp to delete text? that's AWFUL! But classList (https://developer.mozilla.org/en/DOM/element.classList) doesn't exist everywhere, so bleh
No, it is an old comment.
ahh yes.  fixed.
Please use indentation and brace style consistent with the rest of the code.
"receive", "an address list"
MINOR: Refers to .cpp instead of .h in comments.
Any particular reason you have 2 ifs here? I would handle during merge, but I want to make sure you didn't intend something else.
@garyrussell We should probably allow upper-case Uri schemes, just for completeness:

See http://www.ietf.org/rfc/rfc1738.txt section 2.1:

>For resiliency, programs
>   interpreting URLs should treat upper case letters as equivalent to
>   lower case in scheme names (e.g., allow "HTTP" as well as "http").
In the normal flow It won't get here unless initialized. Of course you can call it directly and it will return you what's there. . .
again, don't we need an else clause?
Well, yes. I'd agree with you if this was a private method, but its a public method and therefore can be invoked independently
This comment doesn't make sense to me; where else is a single Map payload parameter assugned to the payload (the if above explicitly avoids mapping a Map).
The From property is not used and not defined in the MSDN docs for query. It should be removed.
I'm concerned about using of 3d party gems like "fileutils" in chef cookbooks.

we should use native chef/ruby or bash methods to perform this operation.

you can use file resource like this:
```
Dir.glob("#{node[:db_postgres][:datadir]}/pg_xlog/*").each do |item|
  file item do
    action: delte
  end
end
```

directory resource, you can delete directory and then create it again
but this way can break postgres and need testing

or you can use bash script to remove this files.
ok sorry about the 80 `word` limit. What I meant is the 80 character limit. I did it in a hurry and haven't completely adhered to the limit. 

> By the way, is it OK to have line breaks like that in .md files? Don't they affect the formatting in some way?

No they don't.
To get the portlethash, you should use the portlethash-{hash} css class on the same element.
yeah this is changing to http://localhost:5000/v2.0/tokens in the new updated client
While you are at it, please also remove this extra blank line here.
The version should probably be 2.7.17-1puppetlabs1 or empty (empty means attempt on all upgrades).
I think dialogs don't need to be inside a section with role region. I've seen that it works regardless it is inside or outside but I would suggest to update the example to avoid confusion from authors
Sam Minnée (with accent heh)
What do they do, do I need them at all?
I've created the visual studio project and first build, and submitted  
it to GitHub. Could someone look at it to make sure I did it correctly?

Thanks,

Eric
Delete all fcts related to deleting shared resources.
Add a short statement about the relevance of this example (e.g. web-service with C++ and Java-implementations)
make clear that these are domains.
This is all of the original files from the .zip archive, just expanded.  This review is just about adding the expanded files to the repo, so that it's easier to track individual file changes.  The actual package updates will follow this review.
How about that?

```
--- a/src/archetypes/referencebrowserwidget/widget.py
+++ b/src/archetypes/referencebrowserwidget/widget.py
@@ -51,13 +51,16 @@ class ReferenceBrowserWidget(ReferenceWidget):
         basepath = '/'.join(url_tool.getRelativeContentPath(instance))
         directory = ''
         if getattr(self, 'startup_directory_method', None):
+            sdm = self.startup_directory_method
+            if sdm.startswith('@@'):
+                method = instance.restrictedTraverse(sdm, None)
+                if method is not None:
+                    directory = method()
             # First check that the method exists and isn't inherited.
-            method = getattr(aq_base(instance), self.startup_directory_method,
-                             False)
-            if method:
+            elif getattr(aq_base(instance), sdm, False)
                 # Then get the method again, but with acquisition context this
                 # time:
-                method = getattr(instance, self.startup_directory_method, False)
+                method = getattr(instance, sdm, False)
                 if callable(method):
                     method = method()

```

I think it does the same and saves some unnecessary calls.

A test for it would be nice.
К чему это предупреждение? Я бы вообще проверку существования картинки перенес выше, и даже не задавал бы никаких вопросов, если ее нет.

Или можно даже лучше сделать — искать все незакоммиченные картинки в `images` с помощью `git ls-files -o | grep images`. Из каждой такой картинки делать дефолтные Title, slug и задавать остальные вопросы, а потом генерить для нее md файл.

Так можно будет сразу скопом несколько картинок заливать.
Como que no? En el archivo /lang/es-es.php hay añadidas 23 lineas
I believe that is what we had on previous platforms as well.  Please display both.
Add some comments for this "fieldsToBlob" object
does the value of isfirstViewControllerViewOverdraggedToLeft really change here? because when you push or pop a viewcontroller, the value only changes during the time of the animation.
comments says remove oldest and the function is called removeFirst, looks like a contradiction? Call function removeLast or something
Noup, but

~~~clojure
(<= 13 age 19)
~~~

looks neat :)