labofsahil/github-event-dataset-2012 huggingface dataset:
When you use this, you just use the "raw" property value (like props.getInt("jdbc.max_conns")). But the ConfigurationService is going to prepend each config with "plugins.<plugin-name>." -- so jdbc.max_conns becomes plugins.rest.jdbc.max_conns. I think this class would be a good place to add that prefix back in. So conflig.getProperty(property) becomes config.getProperty(PREFIX + proprety), where PREFIX="plugins.name."
Additionally, I would put this logic in "util.py" rather than here.
If I change it to just IWalkable, I get:
    Unable to resolve symbol: IWalkable in this context, compiling:(clojure/core/logic/nominal.clj:36)
I see your point.

But I think that listener are not the kind of stuff mergeable.
IMHO If listeners are changed the user should handle this behavior.

IRC This is the expected behavior on hibernate.
Can't we add this map to the class definition and simplify this here?
In my use case user doesn't know groups. 
We could either rename parse_groups into groups and make it public or pass :groups option to compile.
NP: $envs could have been use as a Set rather than an array.
I would suggest not to mix the hash of options from `to` with the one of `:mode`. Something like:

```ruby
to 'cities', :mode => {:unique => [:name, :zip]}

#or even

to 'cities', :mode => [:unique, :columns => [:name, :zip]]}
```
Can you manually edit the buildstats to prettify the easyconfig? I opened an issue to clean up the actual emitting of `buildstats`, see #233.
maybe your version of pkg-config was fixed where you ran the tests? Or it used a cached version somwhere.

Anyway, the demo's will fail if pkg-config can't find the correct libs, so this should be ok now.
That would be cool (although it seems pretty solid as well). Your call (although, perhaps in a separate pull request if you want, so this can get in?). :-)
Going to change the header.
What has been changed here?
Integration tests could be implemented as separate modules. And it should be using tools maintained by a dedicated QA team. I think for this project, we should definitively try to add the short-circuit testing to speed it up. I have not done it yet on sylfide, so I dont actully know how to do it... But that has been the #1 complaint about unit-testing in sylfide, so we do not want every other project to have that. I seem to recall that the problem was related to running with Authorization checks; the short-circuit evaluator does not support Filters or something like that...
I guess we can leave it for next step, but should put it high on the list :)
This function gived possibilty to remove a spécifique event
Методы `finished?` и `has_turns?` можно сделать однострочными.
``` ruby
def finished?
  has_winner? || !has_turns?
end

def has_turns?
  turns_left > 0
end
var a = [];

is faster than

var a = new Array();

change please
будет изменяться?
Name it `main.js` on both concat and min.
added missing ul
To be frank, Qt has a very poor security model to begin with (try spamming your QHttpServer with too many requests and you'll see a nice infinite loop within the event loop).  Given that the Tcp buffer already has this problem (unlimited message length), I do not agree with your argument.  And while it may very well be application specific, my storeBody() method gives the user the option to decide and is not enabled by default. I would strongly appreciate it if you would reconsider your stance.

Pillow appears to not even announce the request until the body is done:
https://github.com/acossette/pillow/blob/master/pillowcore/HttpConnection.cpp#L287

In general, I have never seen a http service that passes on the receiving of the body upstream.  I think the better alternative is to enforce chunking and message sizes.
.NET Provider -> .NET Driver
Hum, copy pasting, even your own code, is bad. That was part of what I did in imajax, don't know why I did that at first, but it isn't used at all. Will remove.
There is no asynchronous work in the APIs anymore. APIs emit an event, and a listener in search.js picks them up and fires an indexing task to the MQ.
What's the reason that this is being removed?
It's part of the API doc (5.5.12)
The tenant server, adjusted the description.
The code is very well written. I have a couple of minor points, mostly regarding naming.

I don't think we should push too hard on having DSLs everywhere. `in` is not a good method name, it depends on having something else around it to understand what it does. So I'd rather make this method `findTestsIn`, or simply `findTests`.

I also think the collector should be a simple abstract class, not a trait, take `global` as a parameter.
Actually, for logging/debugging purposes, it is better to have a proper class instead of using anonymous actors (this way you can override ``toString``)
OK, as I thought. Of course, you're right. I will analyze CH code and try to remember about abstraction. Thanks!
This style is lost and the container titles have too much padding around them:
```css
#content.user article.container div.property-title {
  padding: 5px 0;
}
```

Perhaps improve this one or add one to correct the alignment?
```css
#content.user article.container a.remove-large {
  margin: 13px 10px 13px 0;
}
```
Let's fix that in all providers :)
是这个地方！
Je hebt `healthBar` niet defined
Простите, впредь буду следить, чтоб при ответе не цитировалось предыдущее. 
--
Your sincerely,
Sergey Lebedev.

Twitter: @drasgo
Facebook: s.a.lebedev
ага. понял. спасибо.
@derats сделав расширение класса, как это вы написали, вы объявили `@private` переменную класса Cell. У неё нет ни сеттеров, ни геттеров, поэтому в методах категории ничего не известно об этой переменной.
Я надеюсь, эту тему Рахим затронет подробнее. Но настоятельно рекомендую пользоваться как документацией так и сторонник литературой - в любом случае, абсолютно всё разобрать не получится. 

И вот еще - критерии оценки выложены вполне конкретные, все эти замечания, примечание и пожелания в большинстве своём направлены не столько на выявление ошибок, сколько получение новых знаний.
Лучше уж тогда NSArray возвращать
I am curious as to why this code was split across 2 scripts? Putting it all in the Python script would eliminate the need to leave environment variables defined after script finishes.

Maybe environment variable can be cleared when done? If not, then we may want to prepend BRACKETS_ on the env vars to reduce the chance of a naming conflict with some other app.
Yeah, I feel similarly about the `RACAble...` variations, but I hate to get rid of them since they read so much better:

```objc
RAC(self.foo) = blah;
// vs.
RAC(self, foo) = blah;
```

One looks almost like a legit assignment and one looks like a function call.
Ain't using it, ain't typing it.
Can you make the naming consistent?
Yes, empirical values. It just worked good and helps to approximate a square like look. In fact this seems to balance the layout. As I, following my initial intuition made the width bigger than the height, the channel windows seemed distorted and squeezed.  So instead of fidgeting around with this I kept it like this. But If anyone has a better solution, I'm fine to adopt it.
that would be to simple ;-) and it would still break stuff for users, so they will need to download the huge tar file over and over again..

I was thinking of having the mne-sample data directory tree on the web server; each file gzip compressed + md5 checksums. We would define the version of the sample in a file in the sample data directory. Then, when "import sample" is called, it would compare the version of the sample data with that of mne-python and if it is older update it incrementally (by computing the hashes of the local files and download the new ones if needed)
'engineer' and 'Engineer' were just examples you should not treat them as a special case. What I meant by the 'enginieer/Engineer' example is that we shouldn't consider the case of the letters in the query. That isn't just limited to this example; the user might type EnGineer or ENGINEER or engiNEER but he should still get the same results each time.
Looked like some +\u00ef\u00bb\u00bf was added by mistake in the previous commit
Q has a ncall method, although I'm going to do that in the next iteration to get less code churn.
Here's where the redirection happens.
Syntax error here. '\`' will be enough. Can you check the rules for other languages and see how characters are escaped?
Is this the only remaining unresolved issue with this PR?
Same shit here.
Читабельно еще как!
Само собой в клипборде будет последний. Ты запросил - выдался нотификатор,
что урл такой-то в клипборде.

>
`.more` 應該也算是共用模組?
Drupal development is definitely 2 spaces with new lines. Unfortunately, when we were writing the sdk, we went with tabs instead of spaces, with no line endings decision.
this doesn't look right. are you missing an else {} here?
As above, the ```->``` is not needed here.
Huh ? remove that
Does -fPIC work on 32-bit systems, too? Of course you can also do:

    make CFLAGS="-g -O2 -Iinclude -fPIC"

Instead of adding it to the Makefile.
I don't know. Aren't you supposed to escape things inside an xml file?
Not many people know the conditions under which braces can be omitted. It's clearer to leave the {} in. Most projects do.
It might be good to remove these checks from the manager class.
Ew, good catch. That was a console.log() at one point. I'll fix and squash. :)
Hmmm, what exactly is the use case for these originalX attributes? Whatever they do, can it be solved without the need of these extra attributes?
Ok. I see why you used the class as the ID. Instead, I recommend using the html5 data-* attributes to store data, instead of the class. This will require you change the modal_video.js file as well.
the case is wrong here and in the class_exists check, meaning the autoloading might fail on case-sensitive filesystems
Shouldn't this block happen in the else clause of the try except block.  Generally I favor one line per try block, and I don't see how the new code can raise a KeyError
Why this empty line?
can you replace `*getattr` with just By.LINK_TEXT and a formatted link string?
In the future, it's easier to read long/concatenated keyword params if they're on their own line:

    deprecated_entity(name=attrib, version=TB16_DEFINITION,
                      message="This attribute has been changed from "
                              "`nsILocalFile` to `nsIFile`.",
                      bug=749930)

Also, `version` doesn't need to go on its own line, and bits of code should be surrounded by backticks.
Nit: "change"
Yes, pushed the change to this pull request.
ok

On Tue, Aug 7, 2012 at 11:07 AM, François Thillen
<notifications@github.com>wrote:

> In bridge/ServiceTestConsole/ServiceTestConsole.csproj:
>
> > @@ -35,9 +35,8 @@
> >      <WarningLevel>4</WarningLevel>
> >    </PropertyGroup>
> >    <ItemGroup>
> > -    <Reference Include="ExampleDomainInterfaces, Version=0.0.0.0, Culture=neutral, processorArchitecture=MSIL">
> > -      <SpecificVersion>False</SpecificVersion>
> > -      <HintPath>..\WsdlToDll\Domain\target\ExampleDomain.dll</HintPath>
> > +    <Reference Include="ExampleDomain">
>
> When you add the reference over VS then NPanday add the path to the dll.
> When you add it over a maven reference, then NPANDAY add a .reference in
> every project, which is used for VS to locate the dll.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/loom-csharp/pull/23/files#r1319213>.
>
>
Good catch, I'll fix this and re-run. Note that if there are some certificate issues on the box, https may not work, but then there's no way to know it's safe to download, so your suggestion is safer.
The plan (my private plan??) is to create a mailer class  ... e.g

```php
$mailer = new osTicketMailer($email,....);
$mailer->send($recipients, $msg, $options);
```
The mailer will create a single instance of VariableReplacer and use it for every recipient.  In addition it will help abstract all the for loops used to fire alerts to staff.

Please note that recipient can be just an email address, staff object or client object - so probably not good idea to do ``%{recipient.first_name}`` in all templates.
I will change `clean`.  not sure why I left that in there.  It's more clear, I think, because each part of the tuple is named in the let binding, rather than indexed by `first` and `second`.  Anyway, this was requested by @eightysteele in #104.
Should split this up for readability
@bp323 Wouldn't it be better if we pass in the templates in the callback itself?
That way people wouldn't need to access the `sakai.config.worldTemplates` themselves.
It also gives us more flexibility.

```javascript
if ($.isFunction(callback)) {
    callback(templates);
}
```

```javascript
sakai.api.Util.getTemplates(function(templates) {
});
```
Using negative margin's should be avoided when possible.
The line-height also doesn't seem right (in FF)

I've done some hacking myself and this seems to work:

Add the class `fl-force-left` to `#footer_links`.
Remove 

* `.footer_main .fixed-container.bottom .footer_langdoc`
* `.footer_main .fixed-container.bottom p.footer_langloc`

Add the class `fl-force-right` to the div containing `footer_langdoc_lang` and `footer_langdoc_loc`  + remove the .footer_langdoc class
Add the class `fl-force-left` to `footer_langdoc_lang` and `footer_langdoc_loc`
Please use 2 space indentions setting per our style guide.
Same here: please put the loop into CHash class and just pass number of bytes
This change is to handle "source," or "source=," appearing in the config string -- I think that's what is supposed to happen for the top-level table configuration string, but I'm not 100% sure of that, which is why I'm calling it out.
OptionalInterface seems like the handlers are optional (and they all are)- I'm not sure that fits.

Why is this so hard . . . lol.
We're using Scala's scala.collection.immutable.Seq[T] here which is a parent from scala.collection.immutable.List[t] (a functional List). This isn't java.util.List.
Whenever you're using scala.collection.immutable.Seq (which is the standard) and don't use explicitly scala.collection.mutable.Seq, you're immutable. This is one of the benefits of functional programming.
Here, we're fixing the metadata when doing an update, another bug that was exposed by this one.
Is that your `eval` face?
just a teeeny weeny nit :) can you break this up into multiple lines? not a fan of one liners in js :)
can you prepend with var? :D
I understand that the boolean switch, but how does that affect things?
4 espaços ou 1 tab.
That @Produces was never the correct line, as you should have had text/html in there as well.
I see what this is doing (forcing a full clean so that we get the queryset), but it should be calling `full_clean()` so that it's not so tricky.
You may want to be clear that you're expecting that a *single* function will meet all of the expectations below.
You could inverse checks and split it into multiple lines.

```
if token not equals correct value:
    return False
elif token has expired:
   return False
else:
    return True
```
It is much easier to understand conditions token has to satisfy.
Instead of `USER_CHANNEL_ID` you could simply use `user.get_user_channel` here, no?
Believe me I have read documentation on forms and generic-views several times. 
success_url is used when I call super of form_valid method. Yes, I could just redirect user from there but I think that it is much more suitable to call parent's methods.  

But that is not that important, because I need to set initals and user too.

Hmm, I have got a feeling that we dont quite understand eachother here. I suggest we call it a day and try again tommorow :)
Ok I found out. When I changed interpolation keyword-based it started working. Otherwise it doesn't work. :)
OK, I see that you have been already using nodewatcher templates. Here is spelling mistake I just noticed. :-) There is double *c* in *occured*. :-)
Just one question I asked myself now: have you ever ever in your life used reset button? I have not. Maybe we do not need it? Would this make the form nicer?
> windDirection

This is also display in degrees. You can store that.

> symbol

You can limit this by choices.
This is already there???
Please make sure you respect [conditions](http://api.met.no/conditions_service.html).

Space after `#`.
Also, add a comment after decorator explaining the unit of rate limiting, with possible URL to terms of use, so that later on somebody else can update it, if necessary.
To `home`. Hm, maybe it would be better to move this constant to `views.py`. (We will not use it much there, but it is somehow connected to views.)
You mean that must be in up case?
copy/paste-o
Need to explain what this is for
Style nit: remove this.
Thank you for reviewing this pull request. Since this behavior cannot be observed, would it be possible to elaborate on the issue in question?
Is the double "m" a typo?
This is another comment from me. It could have been done by another person, but I didn't want to create another account.
Can we not have mvc at all and just use a regular http handler or something? Seems like alot of added complexity.
Yep, and that way looks a lot nicer as well, or though the output is still the same. Your way is also twice as fast according to the timeit module.
listen стоит вынести за цикл. Он переводит сокет в пассивный режим на прием соединений. accept ждет входящих
Missing semicolon, on `on_view_created` too.
Correct, why not use `profiler_timeline_row_dec` to access the `height`, `paddingTop`and `paddingBottom`
The suppressing is quite complex, maybe you could skip the `_suppress_updates` bool and only use `_suppress_updates_url`?
oops, I didn't think about that. 
I guess when I need the proper shape, I use the integer indices, ``idx = np.nonzero(mask)``
why the change to http? i always thought git was faster? but i guess i actually have no clue ... :)
Might be nice to pass the full options hash to `#to_s` instead of just the unit.  Also, you can give `options` a default value of `{}` so it's `def ago(options = {})`  That way, `options[:base_time]` is less likely to throw an error if `options` isn't a hash (or an array).
No default is fine with me.  This does seem like something that the user should be required to specify.
Yes.  Good catch.  I'll add that documentation.
Suggest moving these into the .cpp files rather than the .h file so don't have to recompile everything that includes this header file whenever they are changed.  The stuff in the .h file for ExponentialInfo can be hard-coded in without the ifdefs.  Just preserve the ifdef USE_1D_DEVIATE_EXPONENTIAL for the choice in the shoot function for whether or not to use the sampler object.  

Also, at the top of the .cpp file where you have these #define statements, add in some explanation for our choice of why we do or do not define them, pointing to the timings file you have in devutils now.

For USE_COS_SIN, we can make it defined only for icpc by writing:
```
#ifdef __INTEL_COMPILER
#define USE_COS_SIN
#endif
```
As a matter of naming convention, should we think about standardizing on something like Count or Number as a prefix like CountPendingCreditRecordsDeleted or NumberPendingCreditRecordsDeleted.
Consider dropping bundlor support, as spring-framework has done.  Contact me via email to discuss further.
Is there a drupal attribute for the URL? That would be a useful thing to add.
Is there supposed to be a $ at the start of this string?
save the planet: stop hardcoding
あ、いや UserConfig はふつうのARなのでIDを細工せずに /users/2/configs/1 とかで PUT すればいいんじゃないかと思ってるのでした。
そうできない理由は何かありますか？(というのが最初の質問になります ><)
Would it be enough to generate dates using java.util.Date's ctor that expects millis since the epoch?
Good point.  It's unlikely to happen but we should handle it gracefully.
Right, this is now the main error message. I'll change it so.
Oh I see your point. Yeah, wut? I don't see why it can't be that instead.
do you really need to do this check ? I would simply do the ``instanceof`` check
Also, I am actually confused how to request this: can you add some inline documentation about what is necessary to call this endpoint... also there's the [readme](https://github.com/AllPlayers/USARugby-Stats/blob/master/Readme.md)
It is created by `git clone`. We don't need to call to `mkdir`

```bash
$ ls foo
ls: cannot access foo: No such file or directory
$ git clone git://github.com/ayosec/highcarb.git foo/bar/baz
Cloning into 'foo/bar/baz'...
remote: Counting objects: 211, done.
remote: Compressing objects: 100% (111/111), done.
remote: Total 211 (delta 99), reused 192 (delta 80)
Receiving objects: 100% (211/211), 30.11 KiB | 22 KiB/s, done.
Resolving deltas: 100% (99/99), done.
$ ls foo/
bar
$ ls foo/bar/
baz
$ ls foo/bar/baz/
Gemfile  Gemfile.lock  README.md  bin  highcarb.gemspec  lib  resources
$ 
```
Для if/while/etc фигурные скобки обязательны.
Мало тестов. Нет ни одного теста на корректность получившегося XML
Cleanup needs to be done after 'start' has been called. I figured it made sense to use the built in teardown feature. Am I missing something?
This is the WIP that I referred to in my original comment. We will need to uncomment this part when we get chrome and remote debugging working again. Since we expect this to happen sometime in the next few days, I'd prefer not to go looking in the repo for the original code. Also the code in the first part of this function is anything but clean. I just tried to make as few changes as possible, so that when we fix it for good we won't have to rediscover the wheel.
why add this click?
Suggestion: More tests later, I guess. Maybe if we don't write tests for the coding we do right away then have dummy test entries with TODO's? Need to talk about workflow irl
Use of :download: rather than :source: would embed the files directly in the source tree into the generated HTML, which will both provide the version matched with the documentation, and not require network access to retrive when using a standalone copy.  (How this behaves with PDF needs checking.)
is this the same as ``self.capacity = max(3, 2 * self.capacity)``?
To sure what the role of the ``capacity`` argument is, though. Maybe a short comment for slow people like me ;)
You should use `sklearn.utils.train_test_split` to transform this block into 2 one-liners:

http://scikit-learn.org/dev/modules/generated/sklearn.cross_validation.train_test_split.html
When `calc_dual_gap is False` the `(w_max == 0.0 or d_w_max / w_max < d_w_tol or n_iter == max_iter - 1)` condition should be used as the "cheap" convergence criterion.
Why are you adding commented out code?
Possible improvement suggestion.  Rather than the method of recursively diving into setSymbolOverride to find the symbol(s) to override the symbol name of the decl.  Maybe try adding ```char* symbol_override``` to struct Symbol in scope.h, initialisation of NULL in Scope::Scope, etc...   Then in this block of code add ```sc->symbol_override = name;``` and in FuncDeclaration::semantic, VarDeclaration::semantic  ```this->symbol_override = sc->symbol_override;```.  This would mean you can loose setSymbolOverride and the code in the hunk below too - if all only to save a couple lines of code and maybe some wasted processing. 8-)
Why is this value lazy? This value can't change. I think that this value is't lazy enough.
Done, good catch
the name shoud be set_escaped_name.
could you add a space also before the "=" symbol
Didn't use this variable in the gradient - it looks like had one idea then went with another.
May as well delete this extra blank line at the same time.
Could you make flag omitable ?
Is there a window here , where serverStatus could be destroyed in disconnect()?
There is no `is_absolute_path` bundle config
because it needs to bind the the parameters in config.yml, no?
This is not what `Platforms.get()` returns.
```redirect_to root_path``` насколько я вижу, выполняется безусловно, зачем он внутри ```if``` :question:
Can't remember but I think the `LDFLAGS` were needed here.
please don't add useless commented use statements
`hasRange = ".." in rvalue` is a slightly more pythonic way to express this
That's right, Marco. Thanks for spotting.
Thanks for the feedback, I will implement a new utility function.
If we set this to False, we can keep paginate()'s default at per_page=None, honoring other paginator's defaults, while avoiding forced pagination-with-no-pages.

However, this only fixes forced pagination-with-no-pages for class based views. If this is going to change over all, we need to tweak the [init of RequestConfig](https://github.com/bradleyayers/django-tables2/blob/master/django_tables2/config.py#L15).
@christianv @shacker Reminding myself to ask you about this later :)
@christianv Did some fiddling with line-height, but had no luck. For a div to show a background requires that it have a width and height. Perhaps we can look at it together?
I think this was intended for some awards that technically aren't associated with a team, like Volunteer of the Year.
This bit does more than just make the items per page configurable - more options are added (20 and 50). Please remove the extra options so that the default behavior is exactly the same as before this patch. In other words, just remove 20 and 50 from the array.
Should check if 'newName' attribute has been set which indicates the username is being changed at the same time. This will often be the case when renaming Local Administrator.
Again, not important: remove open lines between parameters.
PEP8: line length. Maybe define `within_dist = dist_parent_flat > max_dist` and use in the 2 indexing operations above.
We can't use this function for WPEC, it's for used with WP core functions (the error message refers to WP core version).

We'll need to make our own deprecate function. For now, just use @deprecated phpdoc tag.
Very nice.  Exactly what I was hoping for.  I did not look at the code closely enough.
This removes my workaround:
```
context.getMaximumTextureFilterAnisotropy() || 8 // TODO: Remove Chrome work around
```
Was this bug fixed in Chrome?

Also, this should probably be a parameter since it can significantly impact performance, e.g.,
```
maximumAnisotropy  = 8; // Exposed somewhere
// or:  maximumAnisotropy = context.getMaximumTextureFilterAnisotropy();
// ...

Math.min(maximumAnisotropy, context.getMaximumTextureFilterAnisotropy())
```
it was causing the vertex arrays to be recreated way too much because of the simple implementation. It was constantly toggling back and forth in dynamic examples like simple.czml
are you referring to the partition method? can u elaborate more on this?
K, then what about setting a sensible default path in the config file? The reason I bring it up is because I didn't think to set the harnessConfigsDir in the config and the error message was not helpful.
not read only in android
this is needed due to a problem with port forwarding with adb. in short, if you run a pass for android and then try with iOS it will fail unless adb is stopped. there may be a way to refactor this in a more generic way though - investigating
Indent 2 sp :)
only leave one blank line
`grunt.log.writeln`?
I don't think the order matters much. Just make it look good.
Note that `$(D X)` doesn't have to be used when the ddoc comment is on the symbol `X` or symbol `X` is the name of one of symbol `X`'s parameters when it's a function.
linsweep -> linesweep
instead of parse, "compile" or "transform" (?)
Thanks!

We should eventually fix the code to generate truly unique ids. I hope as part of our cleanup later on we simplify these names further. I'm not sure what the end result should be though. Maybe we want to just call them '_listener1', or '_onevent1'. In particular, it might be extra noise to include the idAsIdentifier if the id for the node was autogenerated.
You mean you don't care about updating the header everywhere? Or which name to use? In case of the former I volunteer to take that task so that we fixed this for good.
LDFLAGS can (in theory) contain other linker flags that are not directories (what you want here i guess) nor not even start with -L (which my split example doesn't cover either)
s/observer/observe
http redirects to https for the whole of openmicroscopy.org
Again, don't do this. If there's a known case that you're expecting (e.g. there's no result for this patient) then code around that with an if/then. Don't catch _all_ exceptions to take care of a single known case.
why duplicating the macro ?

```jinja
{% block root %}
    {% import 'knp_menu.html.twig' as menu %}
    <ul class="breadcrumb" {{ menu.attributes(listAttributes) }}></ul>
{% endblock %}
```
A bunch of these attributes get includes in the inheritance from Resource: see https://github.com/aweber/AWeber-API-Ruby-Library/blob/master/lib/aweber/resource.rb
It's actually tested in the SectionPresenterSpec. I just pulled it to build _something_ to test for this presenter.  I'll actually remove this test, and replace with tests against the methods the presenter implements.
bad guess :)
Missing radix parameter here
Extra space before the `++`
Might want to use `$(changedHTML).find('img:visible').each(function()` here instead to make this more readable
Might be worth checking in the JIRA to see if that user can set their locale to something else and also trigger the bug to happen.
Comment about how these were chosen?
Fixed with commit 8717777
Ah, yes. That was a left over from the first attempt for a fix. I've removed the "if" statement.
Write out all arguments to the callback function.
Another option is use the same style of http://zf2.readthedocs.org/en/latest/modules/zend.form.elements.html
Should this one perhaps be named cronjobs_add since it just appends if not found?
add this comment as a **issue** as i will attent to it in another branch.
@croby thanks for letting us know.
these tests were removed since bundle handlers were removed from the portal
Similar to my comment above, i'd group the imports like this:
```python
# ###################################################

import fife.and.fife.stuff

import horizons.main
import horizons.other.stuff
```
Perhaps we should move this to utils, it seems like a generically useful function
rotate : Math.random(-90,90)+'deg'
inro ke mizanam zamanesh kar nemikone
same comment with 'negative' still trying to figure out a better name (if that exists :) )
why is the tag required, will add later if required?
Please keep the coding standard - do not use fully qualified name if not necessary.
ok...pull your request again..!
//begin < advice < element
Aa, you're right. It should be changed to use the org.jboss.spec artifact:
http://search.maven.org/#search%7Cga%7C1%7Ca%3A%22jboss-ejb-api_3.1_spec%22
This should be changed to:

    "CONSOLE: Toggling downfall (on|off) for world|Toggled downfall"

So we can have:
* `CONSOLE: Toggling downfall on for world`
* `CONSOLE: Toggling downfall off for world`
* `Toggled downfall`
this should probably be islandora_audio_sp_admin_settings
@madebydna oh I didn't know about that. Do you have any articles about best practices for the asset pipeline? I probably need to read up on them :wink2:

How about we call the site file `community.js` rather than `site.js`. I just worry newcomers won't know what the difference between `site` and `application`. But if using the `site` file is a new best practice I don't want to go messing that up. What do you think?
I think this can just be:

```ruby
f.label :rate, difficulty, :value => difficulty
```
Yes I prefer the current way because there is a non-trivial amount of things happening for one comprehension.