labofsahil/github-event-dataset-2012 huggingface dataset:
While we are talking about ExecutionContext, I've seen that in Scala, the
global ExecutionContext is not only an ExecutionContext but a
ExecutionContextExecutor extending also Java Executor.
Java Executor is just a subset of Scala ExecutionContext so this is almost
free. (Scala also provides implicit conversion to ExecutionService)
This is useful when interacting from Scala with Java API such as
ListenableFuture which requires an explicit Executor.
I think it shouldn't be a big modification to enhance our ExecutionContext
with the Executor interface.

What do you think?

regards
Pascal


On Fri, Nov 23, 2012 at 10:00 AM, Sadek Drobi <notifications@github.com>wrote:

> In framework/src/play/src/main/scala/play/core/system/Invoker.scala:
>
> >
> > -  val system: ActorSystem = ActorSystem("play") //TODO make sure this is configurable
> > +  val system: ActorSystem = ActorSystem("play")
>
> yes for Promise actor
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/playframework/Play20/pull/559/files#r2209964>.
>
>
Generally I agree that business logic belongs in store and not in store. I have seen a tendency to add business logic to resource, which is a bad idea. Resource is just one way of exposing this API and should only contain API to internal <=> mapping.

In this case, I cannot pin point why, but I feel leaving this check here makes more sense since this in someways is the API contract.
correct @stof that is what i meant
`%kernel.debug%` is [the default value](https://github.com/symfony/AsseticBundle/blob/master/DependencyInjection/Configuration.php#L53) for the `assetic.debug` setting, so this line isn't needed.
We can event do the API reference here:

    :class:`Symfony\\Component\\...\\ApacheUrlMatcher`
Ok, This tip should probably simply be moved above
解释：
这里and后面的variables和前面的不一样，nginx log里面支持两种变量
其一就是普通变量，其二是只能在log中使用的变量，比如现在的$status。
所以这里是在描述这两种变量，而不是在解释普通变量的什么行为。
路径名
完整路径

该路径名
该路径

其默认名称
默认路径
i would actually do `map (_.sum)` here
Ops, sorry guys, I guess I was drunk :-D
You are right, you can't remove these parenthesis, keep it as it was. Again, sorry to waste your time...
The Wikimedia convention is to have one selector per line.
you should be nicer for the constraint. Nothing in the template require the 1.9 features (you only use features available as of 1.0). currently, you forbid users to use Twig 1.8 when using your extension without a valid reason, and you will also forbid them to use Twig 1.10 when released until you update the constraint here (again with no reason as Twig 1.10 will still work for you as all Twig 1.X releases are fully BC). The proper requirement would be ``1.*`` (and it would have been ``>=1.5,<2.0-dev`` if you were using a feature introduced in 1.5)
This should be `error( "...", 2 )` so it flags the error on the caller instead of this method.
Ok, I'll keep that in mind.
Please don't use such spaces. It's incredible uncomfortable to maintain and to read IMO.
I found it...
Yes, None means use the rcParams['patch.edgecolor'], but "none" means "don't color the edges at all".
@pelson I wasn't suggesting we go through and fix all the variable names... that is a gargantuan task (I'd like to have them all fixed in all the other files in the codebase too). Although it's better style to have variable names that are a little less, shall we say opaque, I am happy for this to be merged when the trailing backslash above is fixed.
Don't you mean 'np.log1p(h)' here?  Plus, let's have this kwarg better handled.  First, let us stick with the convention of setting the default to None.  This will allow for future revisions to treat that to mean "the default behavior", which usually means to grab from the rc-params.  Second, let us take the opportunity now to choose either True/False or string values as valid inputs to the kwarg -- not both.  This has caused so much grief elsewhere.  If you can imagine that other log scales (or even more generally, other z scales) would be desired, then let's go with strings.  Otherwise, stick with booleans.
It's not immediately clear to a reader what this dictionary represents. A sphinx style docstring would be great here:

```
self._actions = {}
"""A dictionary mapping action names to button instances...."""
```

I leave the actual contents of the docstring to you.
I went ahead and added the "Changes in v 1.2.x".
Please use ".hide()" instead of `.css('display', 'none')`;
done, great observation :)
After completion of the "swipe up" for the lockscreen, a user should two things - an icon for the camera and an icon for the unlock. By caveat of needing to find and click the unlock button, you are testing the unlock flow. However, the camera button isn't being checked here.

I'd add checks here to make sure that post the swipe, did the camera and unlock buttons become displayed?
А это разве нужно?
Normally, I try to stay in the .Net convention but sometimes a small letter happens :-). I will try to get all to upcase
What a load of rubbish
Does this div close somewhere?
@christianv I thought it'd be better to include it so you can't be redirected to a html page on an ftp server. Does this make sense or should it be removed anyway?
does this need to be at the root level?
This function can fail, it needs a WT_TRET().
Are you sure ``$accelerator`` has the correct type as ``boolean|string`` is not a sub-type of ``boolean``?
Are you sure ``$this->class`` has the correct type as "object<Doctrine\ODM\MongoDB\Mapping\ClassMetadata>" is not a sub-type of "object<Doctrine\ODM\MongoDB\Query\ClassMetadata>"?
Are you sure that ``$this->tokens`` (of type "null|array") is traversable?
I think this is better :)
`is_production()`?
!が足りない
Denne epostaddressen er ikke i bruk lenger, endre til `kontakt@holderdeord.no` (gjelder sikkert i de vhostene for selve appen også).
If $length == 1 then surely you can just do $text == '<' without the substr?
rotateImage uses Imagick
"All rights reserved" is not needed, and removed from Qt a while ago.
ideally this would go in a `beforeEach` since it's needed for every test
Тебе нужно вывести список тегов, каждый из которых -- ссылка на список постов с этим тегом. Нафига делать `tag.posts.each ?`
This string is user-string: use `"` and make it translatable.
See, because you mixed other things in your pull request, you now have to resolve conflicts with main repository. Pull from it and merge.

(You can fix such things locally, but there is no need to commit them into this pull request.)
This probably fails if there is no data returned in `state`?
What happens if `value` is *not* an instance of `ConcreteType`?

One could imagine a constructor like this:

```C#
private readonly AbstractType value;

public Foo(ConcreteType value)
{
    this.value = new Composite(value);
}
```

In such a case, you wouldn't be able to cast `value` (or `Property`) to `ConcreteType`.
setData -> _setData
events -> _events
it makes no sense to handle it here

now i've tested it:
kill a npc that gives rep to this faction

without this aura: 25
with this aura: 46

this aura should increase the rep gain by 25%

look at the comment: "divided by 2 for some reason (See "donerep1 / 2"" i can't find something related to this, so i think it's an old leftover from old code
Is this actually needed with the changes you made to Spell::EffectBind and Player::SetHomebind?
Ok- so I guess we can merge this and create a separate issue to remove the dual-support from all scripts?
please use ``naming_strategy`` instead of ``naming_strategies`` to be consistent with other services (only the listener is using a plural in the id, and I will probably change it to be consistent)
I'll follow it up with a PR when I'll get the time to find/correct it, it is on my todo list :)
sure, that's the correct question.

You have two options:
1) 
- change the files
- commit as a next commit to your previous (this) commit
- squash the two commits together (using git rebase)

2)
- amend this commit
- use `git add -e` rather than the usual add
- commit again
method vs technique. ideally one name should be used
updated the javadoc with an example. The unknown fields is relevant at OutputFormat level (Pig tuple can't carry these).
I think we can support these for thrift as well. I will have a another patch for that.
This is also incorrect because the header names need to be lowercased *before* sorting. I think the original implementation was the cleanest.
Shouldn't this be assertEquals?
So, if we don't supply the changeset here, we might as well handle this with the EventSupport and an additional Event in API, WDYT?
This file needs obviously maintenance.
I don't take the "[" into consideration, because it lines up nicer when having attributes, like so:
```javascript
["span",
   "text",
 "name", "value" 
]
```
So the attribute is on the same level, and the text is indented two spaces from the quotation mark. There are some inconsistencies in this file though. We've never had a standard way of writing templates.
What's the reason for this change? The original code looks right to me (glyph->offy is an int too).
I noticed that one wasn't highlighted in the original readme so I left it alone. I can add that if you want.
I agree. I've rewritten the code to produce /absolute/path/frozen_executable/path/to/module/module_name.pyc, which seems a bit more sensible.
``` haml
- a = capture do
  | World
| Hello #{a.call.strip}!
```

?
It sends it using the :test delivery method
well, shouldn't it still pop out, to get the appropriate behavior to match what a client (or any other ad, really) would see?
"Death to Israel" would look richer if you used CSS instead of <strong>.  You'll never defeat the Zionists with that HTML code....
I'm pretty sure `boost::shared_ptr` contains its own assertions in `operator->` and `operator*`, so there's no need to add our own (I won't bother trying to comment on all the others).
Inverse variance where the variance includes sky + object flux?  Or just sky?  The reason I ask is that if someone wants to fit models to objects (galaxies / PSFs), and the models aren't perfect matches for the light profiles, then fitting with weighting by inverse sky + object variance can create a magnitude-dependent model.  For that reason many people like weight maps that include sky variance only, and I think we should be more explicit which type these are.
This puts WebSQLStore inside Backbone's namespace/object, but I also provided the two lines necessary to support backward compatibility (where it's a member of 'window').
This is a style question I don't know the answer to, but is it generally copacetic to have an empty __init__.py? Do you at least want a pass here or something to make clear that we know we're not doing anything?

For that matter, why have an __init__.py at all? Do you imagine a future use of this?
a script is more general than an application source file.  what you're calling a script here is an application class source file.  see notes in #1 / #4.

do --app and follow the instructions in that issue (i.e. ResourceUtils.resolve).

then we really should discuss the options for what a good signature is!
use Charsets (notice the final "s")
I'm not convinced by treating aws-ec2 as a default.
Also "us-west-2" is missing; that sort of complete list is not what we want to maintain here. It working for some regions and not others will be confusing...
Yes, I have added a comment for this below, somewhere.
The cast is necessary. Read the documentation for the [Objective-C runtime reference](https://developer.apple.com/library/mac/#documentation/Cocoa/Reference/ObjCRuntimeRef/Reference/reference.html).
Here the default path gets added on construct
The last `match` is useless
Lines from 9 to 16 should be removed, and use only the new route (line 13)

I guess that the `if` was added because, in tests, the database is empty when routes are loaded, and the (broken) code was failing. This is a really bad practice, since it can hide bugs to the test environment.

The `begin`/`rescue` is also useless.
`[*instances*]` needs to be documented as a parameter.
>  Although, Array is pretty lenient...Array(any_object) returns [any_object]...is that what we want?

I think that's too lenient.
Right, we need a separate assertion about `resourceURL` not containing sideloads.
There's no point using static methods in python.  If your method doesn't use a cls or self variable, then it doesn't belong on the class.  Better to make it a module function or rework.

I've converted these to instance methods as that makes it easier to override in subclasses.
Same as in hub, just use a name instead of `opts` and default it to `nil`.
please just check for parent:

if (this.parent && this.parent.registerServicesWithPzh) ...
remove this empty test class
It is a little trick to get rid of content-mixed warning easily: http://paulirish.com/2010/the-protocol-relative-url/
The internal data structure will becomes more complicated in future versions, and the concat method will no longer work.
When talking about that 64 bytes value does cost mean used up cpu cycles or does that magic move with different SPI speeds?
Hier und an anderen Orten: Methoden bitte im PascalCase

private void CheckMailImap(ImapClient client)
Doesn't run I my machine. I haven't looked into why. Its not a priority for me since I feel a lot of the other tests are still a bit out
`Boolean` not `bool`?
![http://funny-pictures.funmunch.com/pictures/Soccer%20Fail%202.jpg](http://funny-pictures.funmunch.com/pictures/Soccer%20Fail%202.jpg)
wow, that's awesome @zghorn (assuming it was Zac, I remember he was working on something like this)
Есть версия конструктора, принимающая parent и child.
@ctweney @raydavis Also had to pass this environment variable to IntellJ's spec task.
This is verifying exactly the behavior we don't want.
This needs to be read-only.
Yup, these stupid little accessors are to allow for Mocks. I really like that the timer is final and is assigned in the constructor, and this seemed like the most reasonable way to do it.
so why isn't the `DEBUG` flag good enough? the jslib just `console.log`'s when `DEBUG` is on
語順に関してはご指摘の通りかと思います。次回コミット時に反映します。
ここでのPowerfulを強力と訳すと、パフォーマンス面の意味でとらえられかねないかと思い、あえて「便利」としました。
Optional: Don't need the blank lines around th import.
Again, consider :download: rather than :source:
Well, forget about it. Other tests need more clean up with that, we can do that later.
Don't use spaces with `()`.
Can you write a test for this?
I haven't benchmarked it yet. I don't know that any one tried this yet. How would you set the fixed rank?
We put `__init__` documentation on the class.
again, I would use "number of samples"
Where does this formula comes from / how is it justified? Is there any online reference or mathematical explanation in the doc to refer to?
I think just putting a `dst = src` at the top of the function would fix the problem.
use readData.size() instead of bufferLength
This sounds like a 3rd party file and the only changes made are whitespace. Please revert these changes.
To bypass the whitespace check on commit you can comment the last line in .git/hooks/pre-commit:
```
#exec git diff-index --check --cached $against --
```
Just make sure you change it back after you commit!
If you want to enforce the format, please do some validation in the setter.
not necessary the ForEach below will call
I'm just ignoring the type for the ones I know are redundant. The generic case (default, down there) does print the type.
might be clearer without else here, or on the line 3 down since it simplifies the indentation, and these are like guard conditions.
Thats correct Jake, leaving it in (note ccdserver does the same)
It was somewhere to hang a breakpoint - should be deleted
Do these gem additions need to be here?
> What about using `this` in `on`, `once` & all ?
> Instead of fighting against prototype it would be simplier for everyone to embrace it instead of working around it.

Nobody fights prototype here, there are no prototypes in 'event/core'. It's unreasonable to require use of `call` on every
invoke of 'event/core' function which is way more common case.

> This `method` function is yet another thing to grok before figuring out how Jetpack code is working.
> It is easy to read, really easy. There is no doubt about that, but when you want to understand what's going on,
> or if you want to debug it and you get a complex stacktrace, it becomes not that easy.

As you can see from the rest of the patch direct use of `event/core` functions is way more common, so seeing
`emit.call` everywhere will be more common and will be confusing than this. How often do you really expect
users to look at this code ?

> So here, instead of:
> `on: method(on)`
> we would have:
> `on: on`
> 
> https://github.com/Gozala/addon-sdk/blob/975b90f51a02d8a0b079c36631d47ce3a428e9fd/packages/api-utils/lib/events.js#L28
i nstead of:
> `emit.apply(null, [ this._public ].concat(Array.slice(arguments)));`
> we whould have:
`> emit.apply(this._public, arguments);`

> https://github.com/Gozala/addon-sdk/blob/975b90f51a02d8a0b079c36631d47ce3a428e9fd/packages/addon-kit/lib/private-browsing.js#L50
> instead of:
> `exports.on = on.bind(null, exports);`
> we would have:
> `exports.on = on.bind(exports);`
> 
> Anywhere else, if we want to use these functions directly, we will have to use `call`.

Anywhere else is much more common cases then those exceptional cases you listed here. Neither API
requiring use of `.call` makes any sense.

> I know it is a con, but pros are: no unecessary functions (no backtrace pollution, no performance issue
> [I was told that methods aren't inlined in current JS engine]) and one thing jetpack-specific less to know (`method` function).


In traits and cortex's etc.. we do wrapping and delegation of each method call anyway and it was hardly an issue. In that regard
there will be no difference and if it ever will be a bottleneck we can talk about how to optimize it.
Was this code taken from somewhere, or is it original code that just implements the formula?
Same comment as above. Can this run in prodution? Use login Manager.
Reason to use @Before : In before , I have set up timeout because I need timeout before every test. The @before annotation runs before each test.
This should go in {% block more_head_tags %}{% endblock %} with the other style :)
yes, i think so.
This trailing comma will break support for puppet versions older than 2.7.8 or so. Can you remove it.
ok, this will belong to the devops track
Paramters look like this then: `__init__(self, matrix=None, scale=None, rotation=None, shear=None, translation=None)`?
The wording here seems a bit off. Maybe "The algorithm processes each channel separately, then combines them using "and, ..."?
Yes, definitely better.
We still recommend in the documentation for people to create their own record before using this matcher (to prevent database constraint issues).  If someone created a record with a nil value & the validation allows a nil value so the disallows would not fail.  If thats the case, we need to create an instance in the database that has something other than the nil value.

That being said, this is the part I would really like to avoid if you have any clever ideas.  I couldn't see any that wouldn't screw up people with database constraints.
I think phone_number should be required
this'll come down the pipe in an update, but what it's meant to do is:

````python

try:
    [(lookup, value)] = kwargs.items()
except ValueError:
   return super_filter(**kwargs)

````
Would you have the time and inclination to make this string translatable? :)

This means surrounding it with our custom translation function ts(), so it would become

link_to ts("Mark for later"), marktoread_work_path(work)

And the same for "Mark as read" below. (It's not mandatory, but it's something we're trying to do, as translation inches ever closer!)
You are missing the `exports xhrGet` attribute.
Someone - *cough* @shunter - should verify this algorithm.
Why create the local `err`?  Why not just `this.error = new Error();`
I'm seeing this pattern several places in the tests, what about making this into a function that takes and runs a codeblock as an argument?
No it's actually correct. Width (inside the parenthesis) is used to specify a maximum width of the label, i.e. it's used when you want to measure the height of wrapped text.
'start' doesn't have to be less than 'end'. setSelection(14,0) produces the same result as setSelection(0,14). If user specify invalid values for start or end, then the behavior is undefined. I think in this case it's better to print out a warning and do nothing instead of capping values and let end-developers think their values are correct.
ok, so the code is actually firing the touch move to the tableView row. If the cell is empty there is no touchmove fired. So the doc update should be to the tableViewRow docs not tableView
Android does not support this
This statement is no longer needed.
Ok, I'm done! Wow, that was a lot of code!
Create a local variable for ${processOptions.workingDirectory}/${attributes['src']}.js so you don't have to keep repeating it.
I can't test this code right now because of a bug in Dart. I am reasonably confident that the setInterval code is correct, but I don't want to switch to requestAnimationFrame until I can test it. I will add another XXX in the code to make that switch once the Dart bug is fixed.

The reason I'm a little bit worried about switching to requestAnimationFrame is because I will need to code some logic to stop the animation when the video is paused or stops. It's not that big of a deal, but it's too much to do in code that currently can't be tested.

It's a good comment.
Why num instead of var? Put spaces around "/".
qlogic mpi supports MPICH_CC, MPICH_CCC etc etc. make subclass of Mpich2 and set MPICH_CCC for CXX usage
как классно это сделать чтобы при сборке в no-op превращался?
It would probably be better as

> $(D popFrontExactly) takes $(D r) by $(D ref), so it mutates the original range.

Personally, I try and avoid referring to `ref` as being by reference, because it can be too easily confused with reference types, which are completely different.

Remove the comma after slicing, and I'd advise removing the words steps and time. Certainly, at minimum, the same word should be used for both, but I think that it works better without either. And actually, I'd probably just reword it to

> $(D popFrontExactly) is $(BIGOH 1) if `hasSlicing!Range` is `true`. Otherwise, it's $(BIGOH n).

Until my changes for `hasSlicing` are in, it's still technically possible for a range with slicing not to have `length`, but it's so nonsensical for finite ranges, that I wouldn't expect it to happen, and it's a lie for infinite ranges, because they don't have length, but it can still slice them. You might as well leave in the tests for `hasLength` in the code for the moment though, just to be safe.
Yah, damn, not sure what's happening. The basic idea is: since there's a need to forward from one function to a function/delegate/method/etc., what is a good design for that?
can you add a note about what this means, e.g.

# Accessing private variables on purpose.  # pylint: ...
It's a multi-line string, indentatin might (not sure) screw things up. There's little that can be done here imho.
ああ、最初のconfigのところでErrorにするわけですね。それならいいとおもいます。
Space before parens.
javadoc of AbstractBehaior states to extend Behavior instead
@doubleotoo will work in his own repository and send pull requests just like everyone else.

Just to make my life easier, I simply committed the initial content directly ;-)
Sold! =D
Is there a way to make this a scope?
If the serialization format is JSON, could use this instead:

            foreach ($responseHeaders as $headerKey => $value) {
                $decodedValueArray = (array)json_decode($value);
                if (count($decodedValueArray) == 1 && array_key_exists(0, $decodedValueArray)) {
                    $decodedValue = $decodedValueArray[0];
                    if (is_string($decodedValue)) {
                        $dateValue = Utilities::rfc1123ToDateTime($decodedValue);
                        if ($dateValue !== false) {
                            $decodedValue = $dateValue ;
                        }
                    }                    
                    $brokeredMessage->setProperty(
                        $headerKey, 
                        $decodedValue
                    );
                }
            }
Yes, good to move this in line
sure

On Wed, Sep 26, 2012 at 12:00 PM, Paul Kalupnieks
<notifications@github.com>wrote:

> In app/presenters/woople_theme/content_item.rb:
>
> > @@ -0,0 +1,8 @@
> > +require 'explicit_delegator'
> > +
> > +class WoopleTheme::ContentItem < ExplicitDelegator
>
> I agree with this last comment. We don't generate rdoc from theme, yet,
> using the :: syntax makes it so the documentation parsing doesn't work
> correctly
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/bigbangtechnology/woople-theme/pull/99/files#r1697133>.
>
>
Why did you choose to do this as a service object?  Do we need to expose this to barclamps?  User seems like a framework thing,
bleh codigo de una sola linea que se resuelve con BigDecimal.valueOf( ... );
* カッコの位置やスペース、アスタリスクの位置などの記述形式は、下記にならってください。
  * http://google-styleguide.googlecode.com/svn/trunk/objcguide.xml
* 「末日を取得」というコメントは、maxという変数をlastDayOfMonth などに変更することで除去しましょう。
Entitäten werden beim EntityManager gelöscht:

$this->em->remove($order->getOriginalDocument());
$this->em->remove($order->getCorrection());
i have rebased this PR to move the requirement check into a private method.

> so what remains to be done is having the route document transform the locale piece of the path into {_locale} and use the locale in requirement rather than default. this should be optional behaviour - or maybe we want to do a LocaleAwareRoute object?

yeah .. but it doesnt necessarily needs to be done with this PR. once it is done, we may want to move that private method elsewhere to make it useable by the DynamicRouter as well. maybe a static method?

regards,
Lukas
Just curious, is this analogous to:

`before_filter :find_target, only: [:show, :edit, :update, :destroy]`?
Will this work for e.g. `const(int)[32]`?
This should maybe be SegmentViolationError. For instance, another way to get this would be when accessing freed memory.
Remember to remove the NO_MOVE flag.
Just `switchToThread = cast…`, without the if?
I just checked it really does not seem to gain us anything. I think the idea of the author was to avoid some static if's in the struct, but they are needed anyway. -> I'll drop it.
Log messages should no longer mention "pseudo" since this is used for both pseudo and real transactions.
The save actions automatically adds this one (unless we explicitly remove it from the save action itself) which is java 1.6 specific. Are ok to have this, or we maintain compatibility with 1.5 ?
Shouldn't

``` javascript
view._removeViews(true);
```

work here?
Should we still advising about this? In PHP 5.3 safe_mode is deprecated and definitively removed in PHP 5.4
You can just write this as prose in Python. I don't think any doc stuff will pickup on this format so feel free to ditch the colons.

The "param" one isn't really needed anyway; I think it's obvious what that is and Python docstring guides tend to say "don't include param lists usually".
Might ask will for the new link for this.
Need a length property, todo?
Why we put slash here? Isn't it should redirect automatically anyway if without it?
remove this due to governance confusion
That would be fine have it as a dependency. All major distros will have it and more up to date as well. 
 
Webworks sdk for linux would have two dependencys OpenJDK and Nodejs

I have had a few people ask me why it does not run and have had to tell them to install java
-----Original Message-----
From: James Keshavarzi <notifications@github.com>
Date: Wed, 05 Dec 2012 12:31:24 
To: blackberry/BB10-Webworks-Packager<BB10-Webworks-Packager@noreply.github.com>
Reply-To: blackberry/BB10-Webworks-Packager <reply@reply.github.com>
Cc: Mike Robinson<badtoyz@gmail.com>
Subject: Re: [BB10-Webworks-Packager] Updating README.md - Adding info for
 prereq step to build node on Linux (#229)

> @@ -7,6 +7,7 @@ This repo contains the code for the BlackBerry 10 WebWorks Packager and submodul
>  2. Install [BlackBerry Native SDK](https://bdsc.webapps.blackberry.com/native/).
>  3. Install [BlackBerry 10 WebWorks SDK](https://developer.blackberry.com/html5/download/sdk).
>  4. [*Windows*] Add Git bin to PATH. i.e. `*Installation Directory*\bin`
> +5. [*Linux*] Build node[v0.6.10], ensure it works on your distro and copy the binary into `third_party\node\linux\` folder

Thanks @badtoyz.

---
Reply to this email directly or view it on GitHub:
https://github.com/blackberry/BB10-Webworks-Packager/pull/229/files#r2326422
You might like to either try ```return MYSQL_USERNAME in [user.name for user in users]``` or ```return any([user.name == MYSQL_USERNAME for user in users])``` if you're looking for an excuse to use a comprehension instead of a for-loop.
It's up to you, but tastypie itself uses pk.
Используй для табуляции либо 4 либо 2 пробела либо таб (лучше 4) но не смешивай их. У тебя по коду то 2 то 4 пробела.
```ruby 
  el1['lat'] = (self.visible? ? self.lat : 0).to_s
  el1['lon'] = (self.visible? ? self.lon : 0).to_s
```
seems better to me :)
I'm guessing the "partially" part here is referring to the fact that the fakeserver doesn't implement the SendMail command?
Square brackets ([]) are used to denote optional properties. Mandatory ones should not have them.
зачем подчеркивание?
Please see https://github.com/appcelerator/titanium_mobile/blob/master/android/modules/xml/src/java/ti/modules/titanium/xml/XMLModule.java#L75
why override the user's options?
Doc quedó desactualizada
remove if you dont need
Does this really free resources? Don't you need to abort something or summat? :D
Due to a weird CSS rule, the applet is padded on the left and is right aligned. Would you be able to widen the applet after I have fixed it?
This change might be typo, isn't it?
I'll rather keep the default value of this attribute  to false by default.
because it is not used anywhere
ok wil change this..
It cannot be like this because it's not an array that is being named "lang", it's:
Dont need to pass args here
Use utils.loadModule to load files from other extensions. We already do this for the event extension elsewhere (whenever we register events)
I have to use

*builder.generator.Container.date_format *

instead

*builder.generator.ListParams.date_format*

in date.php.twig ?


2012/5/2 cedric lombardot <
reply@reply.github.com
>

> > @@ -1,4 +1,7 @@
> >  {% block column_datetime %}
> > -{# Need to implement a config for date formats #}
> > -{{- echo_twig(builder.ModelClass ~ '.' ~ column.getter ~ '|date("Y-m-d
> H:i:s")') -}}
> > +    {%- if builder.generator.ListParams.use_localized_date == true -%}
> > +        {{- echo_twig(builder.ModelClass ~ '.' ~ column.getter ~
> '|localizeddate("medium", "medium")') -}}
>
> Ok for this vars : localize_date_format and localize_datetime_format
>
> ---
> Reply to this email directly or view it on GitHub:
>
> https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/pull/126/files#r760687
>



-- 
С уважением,
Vitaliy Tverdokhlib

Tel.:  +380 50 918 1909
Fax.: +380 57 700 3375
http://www.nitralabs.com
should thix be bb10x for consistency?
Lauseke (<= 13 age 19) palauttaa jo sellaisenaan booleanin.
plain text message regexp : exclude every line beginning with >
Because the system is started with no bundle loaded, and then later in the initialization process, one is set. This service was blowing up when no bundle was loaded yet.
On restart of the pc your fingertouch or pen device always is enabled and must be disabled by clicking the extension, because XInput don't store values by itself.
I think they should be disabled automatically if they was disabled while shoutdown your pc.
Please add some lines under line 1314 like you'll find it for the trackpoint:
        if (!this._CONF_trackpointEnabled)
            this.trackpoint._disable_all_devices();

Add the CONF values you introduced in _loadConfig() and in _connectConfig() too please.