labofsahil/github-event-dataset-2012 huggingface dataset:
I suppose you could do that too. It's certainly a lot easier than what I had in mind. :P

Though that idea never really occurred to me as we simply have another (significantly less powerful) server set up that we use for development. It's just an old Pentium 4 box, but it doesn't need to support a lot of users and this way we can put more resources towards the main server. Though it does cause some problems as we have to make sure the SELinux policy is configured the same way for both servers, along with making sure other variables are properly mirrored in the system's overall configuration.

But if it's handled the way you've done it here, while it does make security a bit more of a pain I don't see a huge problem with having this merged upstream. Though I'm not sure I'd want the feature documented publicly since it could give less-security-minded users a bad idea.
Is the fix in awesome_nested_set.rb a performance improvement? Can i do the same for the rails2 branch of awesome_nested_set?
Commit descriptions are super cool. You should use one next time. :)
Pull request gma/nesta#99 sent for this.  If it's accepted and published to rubygems, I'll do another version of this plugin that uses it.
varför var de bättre att ange pId istället för kundId?
You can see it already used on some CM7 device targets such as the semc series. With the target device isolated, we can remove all the device checks in the makefiles for the subdirs without any problems, too. It makes kanging the code easier for others, perhaps... :P
Sorry - don't recall exactly.
voir également http://www.alsacreations.com/article/lire/563-gerer-la-taille-du-texte-avec-les-em.html
Having the sidebar elements embedded in the menu design area meant that if you turned the menu off, you couldn't display the sidebar elements.

This commit allows the side_bar_before and side_bar_after areas to be printed separately from the menu. Useful if you don't want the menu to show, but you DO want sidebar elements.
N/M, I found some people testing `transitionend` and it not working:
http://blogs.msdn.com/b/ie/archive/2011/11/21/adding-personality-with-css3-transitions-and-animations.aspx#10240357
Shouldn't this be part of the parent module's pom?
Please don't use all caps for variables.  All caps are supposed to represent constants, and can be confusing.  I'd also call the variable SRL_DisableNavBar, but that's just me.

Make that small change and I'll merge, assuming you've tested.  Oh, and check your SRL PMs. ;)
You can just do B := Players[CurrentPlayer].WorldInfo[0];  There's no need to the if statements. :)
I don't think we should redesign the ConsistenHash now, it uses toString of the ring nodes, and appends a counter for the virtual nodes.
Keep it simple.
It would be interesting with a Retry Directive
worth noting is that (this.members ++ that.members) was wrong, because it is a Set you don't get the duplicates that you want
"flow expressions compose" is a statement I think. Reworded: It is possible to compose flow expressions. But anyway, it is confusing.
Please realize that NAnt currently targets .NET 2.0.  Would this work on .NET 2.0?
I'm still having issues with this.  It displays exactly as expected in my GitHub client showing only the change lines, but when it uploads, the diff is not shown correctly.
I think I'll just keep the unpacking line as you suggested. As for the error, I kind of want to keep the manual one so that it's more explicit about what exactly is expected as input.
It's better to just import helpers as h rather than getting it form base at some point I want to remove h from base it causes circular import issues and feels wrong to me.  maybe url_for should be available via the toolkit like I thought it was.
I'll be able to remove all of .ie7 .site-footer because of the better colour definition from my previous comment
I get the use of OBFUSCATED_VALUE in the serialize call, but what's this one for? Is this what ends up landing in our logs? Not sure what this method is used for.
W dniu 26.07.2012 08:23, C Anthony Risinger pisze:
>> @@ -394,7 +396,9 @@ def drawNextYear(self):
>>   
>>   class DateField(Composite):
>>   
>> -    icon_img = "icon_calendar.gif"
>> +    img_base = pygwt.getImageBaseURL(False)
> i would actually prefer to have all images store themselves in `images`. let this be `True` (ie. just call the method, don't add `True` ;-) and move the `icon_calendar.gif` to `public/images`

Sounds reasonable.

-- 
pozdrawiam

Łukasz Mach - lukasz.mach@pagema.net
What was your conversion on this? I think it's SCN/NRV role to cache and avoid round trips by batching using the same method Google did theirs?
return directory.substr(0,17).Equals("pvr://recordings/");
There was a test I couldn't run in isolation without this.  It turns out that `Zip::Zipfile` doesn't have instance method `file` if you don't require `zip/zipfilesystem`.  This was previously done in many of the subclasses' files, but was erroneously omitted from `excel.rb`.  I could just add the require statement to `excel.rb` but it makes more sense to me to have the require statement at the class that depends on it; i.e. the base class.

I realize that I didn't drop the require statements from the subclasses' files.  I'll push an update that does so.
You can use getOutput() instead
I just copied the map from the existing Map example in GXM
How about compression & SSL?
<3 imp
ups, yes, typo, you are right of course!
Problem is that there is no `compile` method in `Twig_SimpleFunction`. So, implementing the interface without being able to have a working `compile` method does not help that much with BC. Do you know of many libs using the `getFunction` method in their own code?
I'd prefer to cast this to `id`, to avoid fragile casts (i.e., if the type of `self` ever changes, it's less code to update).

Also, the indentation needs to be fixed on this line.
2.12345, -3.256... pretty much every double/float constant in the test file. There's probably something broken with the regex, but my brain is too tired to figure it out at the moment.
array_key_exists is poor performance. both versions work the same for `Arr::get(array('k' => NULL), 'k')`
true.. done.
Les vues de choices aussi sont inutiles. Tout comme celle de vote. Tu pouvais pas savoir.
testDebugWithException and testInfoWithException will be included after the release of Modger 1.0.0.0-RC

Sorry about the trouble! :(
This would probably be better as

    shape : {sequence, int}, optional

with the explanation of the sequences in variable description.
``ij`` --> ``image`` sounds OK to me. Although I forgot the details by now. With added explanation it should work.
не надо, когда #602 пофиксится, тогда и будет красиво
тут бы const, или ссылку передавать
Ok. There's no need to change the other messages, it wouldn't be accepted I believe, just this one you're adding now. Thanks.
Same here `assert !...`
What do you think about?

```ruby
self.response_body = include_content?(status) ? " " : ""
```
Ha, that's why I didn't find a mention of it :)
why is the `respond_to?` check necessary?
Right now, i am checking if dir in `[:desc, :asc]`, and if not - set to `:asc`
Should i raise `ArgumentError` instead?
Why raise if the parameter if missing ? That's not the behavior for datetime. Similar behavior here would be to set the value (hour/minut/second) to 0.
Ah, I missed that.   Sounds ok then..., maybe a // about "Don't touch these unless you are (a) God (b) Roland Kuhn (c) have 3 or more reviewers"
This needs to be wrapped in try/except. cblas may not be built.
In my specific case, the error is Timeout::Error, but in reality, in under what Exception would it be okay to leave an Order in an non-completed state when there is a valid processed payment for the Order?  Also, I imagine there are many different Exceptions that can be throw by ActionMailer depending on the implementation of the Mailer.

Please advice on how to proceed - I think rescuing Exception is the best thing to do here because any error from the mailer here leaves the Order in a state where it seems that the only thing that can be done to right the Order is to jump into the Rails console and surgically fix it.
Did not know about that method. I simply did not want to repeat the whole Spree:money.new so many times .
I did update the docstring. Should I expand upon the lines 

    This is a specialized version of sympy.rules.rm_id.
    This is necesssary because MatMul may contain both MatrixExprs and Exprs
     as args.
Don't glue statements on the same line, cf. previous discussions.
Can you give a reference to that comment?  I don't think that it should evaluate when evaluate=False.
It's generally not a good idea to use `import *` in `__init__.py` unless you know that you are only importing from submodules with their own `__init__.py` files (e.g., `sympy/__init__.py`) or if the file defines `__all__`.
Also don't use this variable.
```
Examples
========
```
Yeah, unless you are doing bitwise masking for some reason, the best use of `&` and `|` are as the intersection and union of sets, respectively.  `|=` then means essentially `+=` for sets.  `&=` would be useful if you are narrowing down elements, instead of expanding them.  The `^` operator does symmetric difference (I'm not sure when that is useful).
Also, this needs to be merged with integrate and removed from the global namespace but I suppose it's ok as long as we do it before the release.
It used to be that the code never reached this point, because a Matrix is noncommutative and the code would return early in that case. `matrixify` was never called, in fact it is not even imported, thus I am removing.
This is code style convention. You can investigate other framework files to ensure.
app_dict надо же убить совсем.
в основном в frontik.cfg сразу пишут 
```
import App;
urls= [ '/myapp', App('myapp.py'),
'/yourapp', App('yourapp.py')
]
```
это никак не влияет?
why would we add all an empty list here?  Just wrap it in an if.
Sphinx processes the {} in a :file: directive to italicize the enclosed text, it will appear as diaglog._time in hex_ in the resulting doc
I'm surprised this works at all as request_var should prohibit get/post modifications.
Where comes `$multipart` from? I presume that the check has to be `$is_multipart`
`$this->container->get()` will throw a `ServiceNotFoundException` either way, so this code will never get executed.
The dash in "e-mail" will need to be removed BEFORE this is merged into the develop branch.
Oh and replace "doesn't like it" with "considers trailing commas an error".
Use `class phpbb_json_response`.
Fixtures alone don't solve the problem. There are issues with all of the relative paths in the system (tests being run from /, phpBB being in phpBB/, test files being in tests/) and there being no way to configure systems to use different paths without breaking everything.

This only empties the directory after copying the files to store/temp_ext/, and after the test is run it puts everything back as it was. I use the board for development all the time and do not run into issues.
Globals on top of the method body.
Technically that is. It is perhaps still better to go with "dbal object".
There is no new line at the end of the file when I open it in my editor.
One of the more known tenets or Best Practices of OOD is "Favor composition over inheritance", going back to (IIRC) the Gang of Four book.  That's not to say that you should never use inheritance, but a lot of times composition may make things a little more flexible. It's all about the right tool for the job.

In this particular case, changing to inheritance has made the dispatcher wrapper tightly coupled to the Symfony Event Dispatcher. That's not necessarily a bad thing, but the composition route allows you a little more flexibility. You could (eventually, if deemed necessary for some reason) swap out the Symfony Event Dispatcher for another, with a new one that complies with the interface. You could also mock the dispatcher and pass that in when unit testing etc.

I'm not familiar with the phpBB code base, nor claiming that going the composition route over inheritance route (or vice versa) is the way to go, I just happened to see a retweet linking to this thread and this just caught my eye, thought I'd add a bit to the discussion. =)

Cheers,
--mark
him or her? Glumesc, lgtm, merge it!
am sters fisierul inainte de ultimul push

On Wed, Sep 19, 2012 at 4:24 PM, Alex Eftimie <notifications@github.com>wrote:

> In wouso/games/challenge/:w:
>
> > @@ -0,0 +1,726 @@
> > +import logging
>
> ping!
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/rosedu/wouso/pull/78/files#r1639493>.
>
>
Because it isn't the same as "canPlayType". It returns a mimetype. Renamed to "getPlayableMimeType". Fixed. Thanks!
This should probably be `mojit:dispatch` instead.  Inventing new terminology will only add confusion, in my opinion.
We wouldn't do anything with privileges yet?
Why do you require the id column? Wouldnt' it make more sense to simply put the PK on both environment and project?
@Raynos I'm ok with this change, just wanted to find out if @dominictarr had specific reason for not doing `.toString(32)`.

Could you just replace calls to  `uuid()` in this file to `uuid().toString(32)` instead so it behaves same ?
This might be worth looking into for desyncs.
move store out of view
same comment as for the app.js file, more or less. index.html implies that there's a unique "logout" app, but I don't think there should be
I think ``theano-nose`` should go into the user's regular folder for Python scripts if Theano is installed with ``setup.py``. So it should be in the path already. Otherwise, in the source package it's in ``Theano/bin`` (with uppercase T).
Consider instead
""" We check that the special cases of advanced indexing that use CudaNdarrayTakeFrom are handled correctly"""
No, this slot is actually java.sql.Timestamp, so we should convert the string to java.sql.Timestamp, other than TimestampType as the above example. What do you think?
Oh dear, how I did a such stupid thing. I should be more careful. Thanks, Paul.
I would probably rename this "nextLeastConnectedNode()"
On Thu, Aug 2, 2012 at 6:20 PM, Łukasz Langa
<reply@reply.github.com>
wrote:
>> +class Error(Exception):
>> +    pass
>> +
>> +class UpdateError(Error):
>> +    pass
>> +
>> +
>> +def _calculate_zabbix_templates(role):
>> +    values = role.roleintegration_set.filter(
>> +        type=IntegrationType.zabbix.id
>> +    ).filter(
>> +        name='template'
>> +    ).values_list('value')
>> +    if not values:
>> +        raise UpdateError("No Zabbix integration for this role.")
>> +    templates = [value for value, in values if value]
>
> Deride all you want, a single comma in that context does not show intent clearly enough. A pair of parentheses does just that. If you'd rather see `[value[0] for value in values if value[0]]` it's fine be me too.


Both of your propositions look very ugly. The code I used is not ugly
and there is no ambiguity in it -- there is no way to interpret the
comma in any other way than a tuple in this context.


-- 
Radomir Dopieralski, http://sheep.art.pl
А нет такого API там, чтобы в одну строку?
I was wondering about this syntax. It hadn't became more readable, had it?
Could we be popping someone else's cursor here?
You are right, I'll fix it asap.
Thanks!
Please don't change the indentation in the file
Being explicit is faster then coercion, I suspect that extend is a perf sensitive operation...
Shoot, you're right. It's got to go.
To answer both of you, here are the design goals:
* Meaningful errors when `domManip` tries to fetch a script in an ajax-free build
* Maximum resilience against duckpunching of `jQuery.ajax` and subcomponents
* A documented interface for implementers of ajaxy plugins to implement

I thought all of that came up in the ticket, but some of it might have been in side conversations between me, @dmethvin, and @jaubourg.

And while I could see prefixing the method name with an underscore, defining it in ajax.js better satisfies the first and third goals.
no, it isn't. A balance should be stricken. Up to you here. I think skipping the else is a win on bytes with no readability downsides.
To be uniform. I don't think there is meaning to create ourself function names. I make all the function names conforms to boost.
s/madlib/MADLIB_SCHEMA/
No need for additional style-sheet file. Why? Just edit `style.css`.
you can just pass the whole vector into the ctor to make a copy: new Vector3D(User.Position);
Great comments! These were really helpful, thanks!
And I think we want the description to come before the language, since that kind of makes more sense to me…  I'm not sure I can explain why, though.
This is a fascinating interpretation, and I love that you're investing the thought in this.

As mentioned in #46, the language around the handling of lambdas / methods in sections is not strictly accurate to the intent.  In particular, the results of an arity 1 function call a) must be stringifiable and b) rendered against the context stack.  This makes parts 5 and 6 completely non-applicable (okay, maybe not *completely*, but it does seem unlikely that you'll commonly find something useful to call on a rendered template string in a list context).  The text then goes on to say that the returned data value should be "listified" if not already "listy", and that each thing in the list (i.e. just the rendered template) should be pushed onto the context stack.

The failure points in the current verbage (in `sections.yml`) seem to be:

  * The return result from an unary function call is described as "data".
  * Calling `lambda.y` seems to have useless semantics.
  * The return result should be pushed onto the context stack (and hence, never interpolated).

In practice, the desired (and codified) behavior of unary methods is the same as for lambdas: specifically, when encountered, they should be called with the raw content of the section (as per `sections.yml`, point 4), the returned value rendered against the current context stack and delimiters (as per `~lambdas.yml`, ¶3), and the rendered value should be immediately substituted for the section tag (as per `~lambdas.yml`, ¶3).  Dotted name parts following such a function call have undefined behavior.

Furthermore, the "listification" of data elements doesn't happen until after name resolution (which includes function invocations) has completed.  So while `'x': 'foo'` and `'x': ['foo']` are equivalent for `{{#x}}`, `'x': lambda: 'foo'` is equivalent to `'x': ['foo']`, not `'x': [lambda: 'foo']`.

I hope this helps clear things up.  If not, please feel free to say so.  :)
If you define `site_packages` as "#{lib}/#{which_python}/site-packages/", then they will be linked into the top-level but also cleaned up on uninstallation.
Do we need these wrappers for `macosxsdk`?
Does anything depend on the CLT specifically?
Wouldn't it be most logical to have `xcode_version` return `nil` when Xcode is not installed?
Let's avoid assignments inside conditions.
thanks though :P
It seems that this will always return while evaluating the first `j` and `i`? Is this right?
Most things should be Wrapped() before grabbing data to send to the user, as that's the primary (though not sole) method used to ensure that data access is restricted to those with appropriate permissions.
... but now this isn't a good sentence?
I relent. :)
run-time -> runtime
Rule #1: don't check in commented out code
Exactly.  Gets problematic.  One option is to have withPayload as being documented as setting a UTF-8 string.  Probably want the ability to set the payload to a Buffer object as well  utf8 is the default encoding for a lot of the node methods.

```
/**
 * @param {string|Buffer} payload A UTF-8 string or a byte buffer
 */
withPayload = function (payload) {
  if (typeof payload == 'string') payload = new Buffer(payload, 'utf8')
  ...
}
```

Throwing an error doesn't seem quite right, because this is only an issue for HTTP/1.0 servers.  :-/
Newer versions of Ohai support platform families.
That said, media's properly an object, not a string. So it should probably be either `{}` or null. Using `{}` would remove the need for a lot of `if (this.media &&` in subsequent code.
I now realize that this is now inconsistent with deletePath. Can deletePath delete either a file or directory?
Although it's a pain & a bit edge-casey, this should probably go look if there are any other remaining bindings for the same command.  Otherwise it may show blank when there's actually still a shortcut registered.
Actually, one note about the code above: I think the new check can get simplified to `if (parentMenu === this) {`
Now that the looping is happening in `_getOffsets`, we don't need to declare and pass `rangeResults` in as an argument from here anymore--`_getOffsets` could declare it and just pass it into `resolve()`.
Yikes, there's a pitfall for ya.  Does our official tabs/spaces user story have anything about auto-detection or being smart about using the surrounding lines?  If not, it'd be good to add a note or file an enhancement about that.
NIt: indentation off by 2 chars here.  I think there's a tab at the start of the line that's throwing it off.
What happens is that firebug calls toString() on the ceEditor property of window when inspecting objects, and thus triggers the meme to pop up.

There's no real code usage, it's just that the existence of the property makes object inspectors trigger the code.
I'm not sure if we should _ever_ throw an exception, because it also breaks the IDE in the client, or is this exception handled somewhere in a clean way?
Why not keep the dispatch_table in this server's state, instead of the app_env?
Why did you unrequire all of these?
``var_dump`` ? really ?
I tend to only use `block_given?` if I'm also calling `yield`. If I already have a `&block`, I shouldn't have to call `block_given?`.
does this comment need to be addressed?
Oops was probably optimistically thinking of the wrong kind of group.
Ignore.
On Oct 23, 2012 3:25 PM, "Amelia Sagoff" <amsagoff@gmail.com> wrote:

> Can we default it to the group id in brackets?  Thats a field feature
> currently
> On Oct 23, 2012 3:22 PM, "Daniel Roberts" <notifications@github.com>
> wrote:
>
>> In corehq/apps/groups/models.py:
>>
>> > @@ -96,8 +96,6 @@ def get_users(self, is_active=True, only_commcare=False):
>> >              return users
>> >
>> >      def save(self, *args, **kwargs):
>>
>> You can just delete the function, right?
>>
>> —
>> Reply to this email directly or view it on GitHub<https://github.com/dimagi/core-hq/pull/133/files#r1920314>.
>>
>>
>
This is the superclass, right? I think Biyeun intentionally left it open for the subclass to decide what's taken into account when getting reports. Admin reports for example aren't dependent on domain. I'd just pass this by her.
My way works fine too...
That would be a better idea, but rather than a static variable, can you think of any reason not to use BUILDID_PUBLIC from Common.h? It would help clean up at least one similar structure in ItemInfo::setLinkInfo() too.
`page` and `layout` using `read_yaml` too
re https://github.com/mozilla/butter/issues/1045
Let's just make this `dialog` instead of `_dialog` for consistency's sake.
You're hacking this yourself, or putting a new version in?
Looks like the `utf8` argument was dropped here. Any reason?
Can you make this a const to avoid magic number syndrome?
Having test setup steps is fine just so long as they are automated.  I was mostly concerned that removing the snapshots from aws-blue (which has just happened) might break the tests, but if they're running OK with VCR atm then that's OK.

We are quite a long way from being able to run these tests in isolation (ie. without VCR) so I would not worry too much about automating that test setup right now.  This is fine.
@ginko according to @restebanez this validation code is not needed, so we should remove the commented-out code.
@plusjade I actually tried using `@options.hg` originally for the if statement, but it appeared that when you enter the scope of `Ruhoh::Friend.say {`, we couldn't get access to the `@options` object.  So, I tried putting it in a local variable and it worked.  I'm using Ruby 1.9.3.
I added it cause things were still pointing to it at the time I started hacking and by the time I was done I didn't mind it so much. The default should always be `/usr/repo`, but I can see a user editing a URL `/proj/:id/edit` to remove the edit -- and 404 is not ideal.
Remove this line. Don't always enable the text/html content-type by default. There are still some risks when removing comments with HTML page.
ngx_event_find_timer_pt -> ngx_event_find_min_timer_pt
Good catch. Fixed in e515858
Why did you decide to wire the function under a given condition and not in the other PRs?
Should be 
        X = Y[1:n, :]
I say no - this is one area where success messages are useful so you can see if/when someone logged in as you.
This is actually coming from the Compute API -- it was renamed to 2 from 1.1.
Yeah, the notification is a hack to invoke -[RKHTTPRequestOperationLogger HTTPOperationDidStart:] and emit logging statements about the newly prepared request.  It should be re-engineered somehow.
Why you use `trigger_error` instead of throwing a exception?
Please use camelCased names for the variables
Have you tried styling the container as a table cell? That can help you to center horizontally and vertically if that's what you're after, but it's fiddly.
It looks like the default is 'fill', not None
This isn't quite the right form for the "Returns" section - it should look just like Parameters, e.g.,

```
coord : SphericalCoordinatesBase
    A new coordinate object of the same class as this is called on, e.g. 
    ...
```

Right now it's not rendering correctly in the API docs.
Maybe add a sentence or paragraph either here or in the last section that makes it clear that these "for the impatient" instructions apply specifically to the "direct pull request" option?
Isn't it better to use constant and frozen array for them?
If nScriptCheckThreads == 0, there is some special code that just runs the script validation inline, instead of pushing it to queues.

nScriptCheckThreads == 1 shouldn't ever happen - there's some code that turns it into 0 if set to 1.

If nScriptCheckThreads is higher, nScriptCheckThreads-1 actual separate threads are started. When the main block processing thread is done with its normal tasks, it joins the worker thread pool temporarily, becoming the N'th worker, so there are always N threads working.
Renaming it will let GitHub show the changes properly too ;)
Okay ill move it to emesene.py with a conditional statement. No it has to be hard coded to the actual path.

-- 
Josh Fradley
Sent with Sparrow (http://www.sparrowmailapp.com/?sig)


On Sunday, 5 August 2012 at 11:24, Riccardo wrote:

> > @@ -0,0 +1,14 @@
> > +--- /Users/Josh/Code/Builds/Current/emesene/emesene/emesene.py 2012-07-10 11:06:04.000000000 +0100
> > ++++ /Users/Josh/Code/Builds/Current/emesene-osxchanges/emesene/emesene.py 2012-07-12 08:31:42.000000000 +0100
> > +@@ -21,6 +21,11 @@
> > + import os
> > + import sys
> > + 
> > ++sys.path.append("/Applications/emesene.app/Contents/Resources/gtk/inst/lib/python2.7/site-packages")
> > 
> 
> 
> i think you could modify emesene.py directly e.g. if os == darwin then append that stuff to the path
> 
> also: can it be written as a relative path? i don't know if we get some real benefit since i don't know if you can rename the .app, but if that's the case that could actually be better
> 
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/emesene/emesene/pull/1436/files#r1308079
> 
>
ship_type_id
yes, i think we talk about type in this place. regarding mixed collections, we will have to add the option to have a list of type children in a collection definition. so i wonder if we want to do the BC break once and say we have 1-n type children rather than a type attribute. or do we want to allow the type attribute as well for the simple case of single type collections? though i feel this could become ambigous and is rather confusing.
Space after the `if` before the `(`.
I've not seen the current appearance but suspect we're very pushed for space. Maybe a similar approach to the notification bars that pop up when the user needs to log in? E.g. "Load my password database (Login to KeePass)" although I suspect that is too long so...

It might be good to abstract the detail of whether KeePass is locked or closed at this point (maybe exposing the detail in a tooltip?).

So just "Load my password database", "Load my KeePass password database" or "Load my KeePass database"? (the latter might not be quite so intuitive for new and forgetful users).

I'd prefer to have "Login" in there somewhere rather than just "Load" but we might have to sacrifice that for the sake of space.

Also bear in mind that some other languages are more verbose than English so we shouldn't try to squeeze out every available pixel of space ;-)
:-) 
It seems that the userdict was not loaded anyway, I'll have another look at it later.
:( still not good code
just a bad copy/paste, I'll fix
These should go in a macro for future re-use.
Oh, nice catch. This was from when I was trying to do `::{slug}::{locale}::lock` for the key, so I needed to get the parent's slug. I'll nix that where it happens.
This needs to be a relative URL. `../img/...`
Oh wait. So the second entry is the value. The tuple confused me. I just expected a hash. Sorry.
Ok, so lets use that and test that our code behaves correctly in such a case.
we have a [`clearfix`](https://github.com/mozilla/zamboni/blob/master/media/css/mkt/reset.less#L533) if you want to use that
yep, [here](https://github.com/mozilla/zamboni/blob/master/mkt/templates/mkt/base.html#L36) we say `html-ltr` for left-to-right languages and `html-rtl` for right-to-left ones
Thanks. That was probably due to an over-active Eclipse prettyfier. I will reset this line to no-spaces.
заморозь на хеш, або скажи хай теги чіпляють
і на майбутнє так для всіх бандлів роби
I can either replace characters by character(s) or replace element(s) by elements.
The first solution makes more sense to me but also implies some little changes in many places

 * update every existing message to be consistant
 * maybe use pluralization in the translations

What do you think?
the code  above could be moved to a setupBeforeClass static method
I vote for removing this method, to avoid that a bundle remove the config set by the end user
Please remove the priority for the encoders. The order of these 2 is irrelevant (they are handling different formats anyway) and I don't see a reason to make them use a high priority.
@stof : really good catch! There were also a second issue, basically I've added a way to have droppable containers (with a dynamic name), in order to avoid running different cases with the same (as they might have the same class name).
Oh, duh. I didn't notice the controller name. However, hasn't the convention (mostly) been to have controllers be plural? That would eliminate the namespace problem.
This must inherit from Post and shows up in the main page.
If the duration is 0 `UIView animateWithDuration` will do exactly what you've accommodated for there so just type this as the `UIView animateWithDuration` method and include the animation block in there.
It's detail, but type should be `NSInteger`.
Rather than mutate the factory object, I'd rather add the sender and/or event as arguments, as in my example above.
Style:

```ruby
build(
  :invitation_with_group,
  event: event,
  invitee: group
)
```
What's with this magic number? Is there a cleaner way to do this? Is this basically the maximum number of threads we'd have?
Why not just

    $userMap->save($user);
    $user->isPersisted(); // true

As objects are passed by reference it should do the job.
Why not `Tags` instead of `tags` ?
Is there a reason to use a real Record here, rather than a stub?
This is confusing. You have GroupListView and GroupView. You also have GroupList collection and Group model. I'm not sure why you are setting model on the GroupList constructor prototype to point to the Group model constructor. You never use it.
This isn't needed and should be removed. It appears that this was added for debug purposes and fails on Windows.
Yeah, I'll change that. I added the message before I understand that all usage of the String was the same.
Not sure how much of an issue this is, but as far as I can see the internal maxOutboudMessages is not actually decreased when close() is called, so it looks like it may still be possible to have a "Too many open outbound writes" generated, although I am not 100% sure (and even if this is the case I don't really see what could be done about it).
If there have been no management API changes in this subsystem since 7.1.3.Final, this change means you need to do a micro-version bump.
Why remove support for the '~>'  dependency operator? (Rubygems uses this)
Questo file non è da eliminare...
Is there any function up there that have dressed this query of `Keyboards[_baseLayoutName].type` or `Keyboards[_baseLayoutName]`?

Use that instead if there is any.
I'm fine with the mechanism to pan between those. Opening the browser is a good thing while we don't have anything else, let's remove Google from the comment and let the browser app decide.
Let's discuss this in person then.
The device storage api hardcodes "/sdcard/music" in lowercase.  The /sdcard filesystem is VFAT, I think so probably alwas case-insensitive, but we should probably use "music" as the directory in media-samples and on /sdcard, to be safe.

(Device storage will also be changing /sdcard/DCIM to /sdcard/pictures, soon so we'll be lowercase consistently)
Oups. I forgot about it. Any other name would work, just want to avoid what seems like a typo.
You may want to add a comment here asking to be in sync with the other reference of that.
ok, i don't know it, i remove them
for ( ... ) {
          for ( ... ) {
            ...
          }
        }

You are missing the first set of `{ }`.
It is a matter of style and aid. Due to there is no way to indicate module-level private members in JS, I use a _ to distinguish. This way, in addition with reveal pattern let anyone to distinguish between public interface, module-level private members and function-level variables without looking for declaration.
Do we really wanna keep this commented line?
Take some!
Name your parameter 'contact' instead of 'c' and use that?
There are *'double quotes'* missing in `data-l10n-id`
I am not sure how I ended up looking at this but why is `assert.isTrue` used over `assert.equal` ? Error messages are quite a bit better.
and of course by "type" I mean "typo" :)
Why don't we use the correct BB here and so we can avoid spin library? in other PRs maybe...
contactButton
Ciao @francescoagati 
I'd like to see `list` instead, and spaces around `=` …just because it's Ruby :)
@brunomachado, it looks weird that the legend is so far away (search form in between) from the title since they look so close to each other in the design... It would make more sense if they were closer in the HTML as well.

Also, it doesn't look like a paragraph for `<p>` to be used. A `<span>` would be a better choice.
looks like load_json will return nil if the file isn't json, is this overkill?
you can use `findById` here and just pass in the `_id`
@mpapis should be ok now, did a plain `ln -s`  (those env vars are superfluous at the moment)
Catching `Exception` should be done as a last resort. Catch specific types of Exceptions first and place error apps for them, and then finally catch `Exception` and log what type of exception it really was, since we aren't expecting any other types of exceptions to be thrown by this point.
Log and print if you were going to use the SSL port but changed your mind.
Verify we get only a single option, correct for the user.
What to do if the access is something else than Joomla? ACL in Kunena is handled by plug-ins, and anyone can add their own ACL.
Why test with in_array(0,$array) it's done just after ?
Not nulling threadID works too though?
This doesn't look specific to the force scheduler at all. It seems like this could profitably be moved to the base class. Perhaps `addBuildsetForSourceStampSetList` or something? (That name seem unnecessarily long.
I can't make it compiled successfully if using v110_xp. Instead, it works on both vs2010 and vs2012 if i changing v110_xp to v110.
Any ideas ?
a simpler composer valid file would do
our function naming scheme is ThisIsAFunction
axis should always turn around the z-axis. If needed, please rotate the coordinate system

so axis should always be <axis xyz="0 0 1"/>
Actually this documentation no longer seems correct. The user can provide their own slot name in the `slot` argument.
Notice that the previous comments wasn't true: the JSM scope is returned in any case, even a second argument is present.
Passing an empty object helps to keep the SDK's module scope cleaner, otherwise without the second argument the `EXPORTED_SYMBOLS` are imported in the current scope (previously was the loader's scope, see [bug 683217](https://bugzilla.mozilla.org/show_bug.cgi?id=683217))
Maybe we would should read this async as well (XHR).
Not sure what you mean, could you explain a bit more? The only issue I can see is about context object, but that was just a quick example, can be easily fix. But it seems you're referring to something else. I'm not sure what you meant with "exposing our method `f` to content and overloading method set by the page itself"; but especially the last part about expandos is not really clear to me.
I'm not following you here?
In any case, according to this documentation I changed `x-default` to `m-cfx`.
And used a more specific xpcom name `@mozilla.org/cfx/clh;1`
an explicit 10s timeout for checking heartbeats is nice.  This allows us to test and in cases where nodes we depend on are not reachable down, we're not running a bunch of simultaneous http requests.
Good point, thanks.  I've updated the branch, however I've returned /selinux as standard instead of an empty string, as it was the standard fallback.  By changing the source /proc file used too, we've got much better compatibility (e.g. on RHEL 5).
One of the things I worry about is the following case. The in_new_thread code could trigger a GC (it uses methods accepting a GCToken), so we might have moved the this pointer when we wait for the lock.

We probably need to copy 'this' into a variable and mark that variable as OnStack so we handle that case correctly.
What are these ones for? No comments here to let us know.
Not a big deal, but: I used to have `mocha.reporters.Base.slow = 200` in my tests, but now I have to capitalize it. @visionmedia, do you think it's OK to break the syntax (without bumping the major version), or is this a public API we need to maintain?
Actually, I don't think it should have the ```AS "bar"``` part. It should likely not include an alias, right?
According to the coding standards, you don't need import "Code within the current namespace or subnamespaces of the current namespace"
Correct, but the init() _is_ always triggered with ModuleManager as argument (see the Init listener here: https://github.com/zendframework/zf2/blob/master/library/Zend/ModuleManager/Listener/InitTrigger.php). As a module, you do not need to rely on the manager, but if you do, you need to be sure there is a module manager. That's why it also makes sense to make the argument no longer optional.
war besprochen
I am a bit loop from medicine right now so forgive me if this isn't obvious but...why would Spout run plugins in proxy-mode?
I don't see any point in quoting these
is it expected to use the ResultCache implementation for the hydration cache too ? Currently the place using caching in the ORM are using separate cache implementations
I lol'd hard X°D
Why not extend AbstractClassMetadataFactory? This gets rid of most the code!
you should use strict comparison whenever possible, and I think it is possible here
We should avoid `eval` where ever we can. In this case we could do something like this:

```ruby
redirect_to self.send("#{@comment.commentable_type.downcase}_path", @comment.commentable_id)
```
Oh, right, this is a Sentry client. For some reason I thought I was reviewing a patch for Sentry itself.
a VO makes a lot of sense, and this is the vsc namespace. when shipping the schema we should clarify a few concepts that are used in the VSC 
a VO is a group with a protected name (the concept of a VO itself is borrowed form the grid world)
Actually, they are described in the root readme.
Same with this. expect should be passed in to the function as the second param, no?
I think we should bolster the test at this point to test that the count of results is 0 too. The locator should be fairly basic, something like `ul.mw-search-results li`.

The reason is that assert 'false' and 'not' can easily give false positives of for example the locator of the search results area changes then the test will think that it's not present because the same ElementNotFoundException is being thrown. 

Admittedly this could occur with my suggestion too but at least we can strengthen the protection against this occuring on two fronts.
undeclared `shiftRegister` var
Have you tested this out yet?

I believe you'll want to do something like `TourOfHome.collect(connection.get("/listings/tourofhomes", arguments)`, otherwise you'll be returning an array of Listing objects instead of TourOfHome objects.

Make sure to ensure that in your test. (something like: `results.should be_a(TourOfHome)`)
In the model for Event, I specified the length of the slug to be the same as the length of the title, so the slug should always be ok (slugify just replaces characters).  I accidentally left this out on participants though.
Why does ensureRendered need to be called?
I don't think we need that here.
Haven't tested, but would `if ($.trim(String(validator.guess).replace(/,/g, "")) === "") {` work as well? Ah, I see this you got this from `handleSubmit()`. Please factor it out into a function (like `isAnswerEmpty` or something).
Why bother having `unit` as a keyword argument if it's going to throw an exception if it's not provided? Seems like it would be better to use a positional argument instead—that's how I would probably use it anyways!
Left this untouched as I fix it in a subsequent patch.
I tried what I thought was the default admin:
admin@example.com
password

didn't work for me
I was following the instructions from
http://activeadmin.info/documentation.html
Ok, I tried something, tell me what you think. :ok:
Hum...

This is the same problem as when we try to serve a simple file that does not exists (without the rewriting stuff).
We delegate file serving to another middleware/code.

If we check file presence here, you should check file presence everytime then.
Weird. If I uncomment line 3 , I could not  use the constants NUMBER_OF_LABEL_SIZES...

Maybe it was just a spelling mistage....stupid me
Prefix with `_locator` and any reason for the new line separating this locator from the existing ones?
Missing comma. :)
No, that's a very important part of the functionality.  This patch totally won't work without it.  :)

doh.  fixing.
Seems like that is less clear, to me: you need to think for a little while to figure out that it actually does the same thing, and wonder if someone accidentally dropped the rest of the functions that `->` should have applied to.
I originally wrote it that way.  The problem was that I wanted the exception to very clearly explain what the problem was... that it was a type failure.  This meant that, if the caller could just pass in any predicate function, they'd also need to pass in some sort of exception formatting string so that the failure error would make sense.  I am perfectly fine with that approach and would have no problem changing it back to that, but @nicklewis was not a huge fan of it when we paired on this.
I'd like to see a test in here that properly verifies that a user that can't see a different user will only see the publicAlias and nothing else. We'll want to do this for several scenarios.
Sure, it's been added.
I'd prefer to get this into a io.js util function.
This is failing for me on my local machine, with the following error:

```
npm info it worked if it ends with ok
npm info using npm@1.1.59
npm info using node@v0.8.8
npm info preinstall OAE@0.1.0

> OAE@0.1.0 preinstall /Users/nicolaas/Desktop/Sakai OAE/ui/Hilary
> npm i -g mocha

npm info it worked if it ends with ok
npm info using npm@1.1.59
npm info using node@v0.8.8
npm info retry registry request attempt 1 at 19:22:40
npm http GET https://registry.npmjs.org/mocha
npm http 304 https://registry.npmjs.org/mocha
npm info install mocha@1.4.2 into /usr/local/lib
npm info installOne mocha@1.4.2
npm info /usr/local/lib/node_modules/mocha unbuild
npm info preuninstall mocha@1.4.2
npm info uninstall mocha@1.4.2
npm info /usr/local/lib/node_modules/mocha unbuild
npm info preuninstall mocha@1.4.2
npm info uninstall mocha@1.4.2
npm ERR! error rolling back Error: EPERM, chmod '/usr/local/bin/mocha'
npm ERR! error rolling back  mocha@1.4.2 { [Error: EPERM, chmod '/usr/local/bin/mocha'] errno: 50, code: 'EPERM', path: '/usr/local/bin/mocha' }
npm ERR! Error: EPERM, chmod '/usr/local/bin/mocha'
npm ERR!  { [Error: EPERM, chmod '/usr/local/bin/mocha'] errno: 50, code: 'EPERM', path: '/usr/local/bin/mocha' }
npm ERR! 
npm ERR! Please try running this command again as root/Administrator.

npm ERR! System Darwin 11.4.0
npm ERR! command "node" "/usr/local/bin/npm" "i" "-g" "mocha"
npm ERR! cwd /Users/nicolaas/Desktop/Sakai OAE/ui/Hilary
npm ERR! node -v v0.8.8
npm ERR! npm -v 1.1.59
npm ERR! path /usr/local/bin/mocha
npm ERR! code EPERM
npm ERR! errno 50
npm ERR! stack Error: EPERM, chmod '/usr/local/bin/mocha'
npm ERR! 
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/nicolaas/Desktop/Sakai OAE/ui/Hilary/npm-debug.log
npm ERR! not ok code 0
npm info OAE@0.1.0 Failed to exec preinstall script
npm ERR! OAE@0.1.0 preinstall: `npm i -g mocha`
npm ERR! `sh "-c" "npm i -g mocha"` failed with 50
npm ERR! 
npm ERR! Failed at the OAE@0.1.0 preinstall script.
npm ERR! This is most likely a problem with the OAE package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     npm i -g mocha
npm ERR! You can get their info via:
npm ERR!     npm owner ls OAE
npm ERR! There is likely additional logging output above.

npm ERR! System Darwin 11.4.0
npm ERR! command "node" "/usr/local/bin/npm" "install" "-d"
npm ERR! cwd /Users/nicolaas/Desktop/Sakai OAE/ui/Hilary
npm ERR! node -v v0.8.8
npm ERR! npm -v 1.1.59
npm ERR! code ELIFECYCLE
npm ERR! 
npm ERR! Additional logging details can be found in:
npm ERR!     /Users/nicolaas/Desktop/Sakai OAE/ui/Hilary/npm-debug.log
npm ERR! not ok code 0
```

We should probably sudo that command.
A stored regex would be about 5x faster according to benchmarks
Hm, ok, I'll think about that.
Any reason why this is ``>=1`` and not ``==1``? 

(And, if this should be changed, the comment about Scala 2.8 can be dropped as it is no longer supported).
> What you're saying is the "Already seen" code will fire (if all sections already seen) and so it won't process but also won't mark as done?

Yes, mostly. It won't process until the mask is reset on the next version change for this status entry.

And since the version on the now/next tables only changes when the next broadcast starts, this means that the scan will be stuck until then.
update the docs for the new stuff
Why casting to string?
Еще бы проверку добавить, что бы символ на такой же не заменялся.
О чём вы? Зачем на вход mutate подавать hammingDistance?
Переменная `prc` содержит проценты, а не количество элементов для изменения.