labofsahil/github-event-dataset-2012 huggingface dataset:
I think it should change, yes.
parseInt should include the base.
ie: parseInt(port, 10);
These assertions need to also verify that the thumbnails are hidden from unprivileged users.

For example, it would need something like:

```javascript
if (!canLoggedIn) {
    assert.ok(!users[loggedinUserId].thumbnailUrl);
}
```
to ensure there is no privacy leak.
Pushed a workaround.
I've followed up on @simong 's comment
Note for future reference: this should be split out as well once we re-organize the content API.
+1, except I think it's better not to execute. We could create multiple upserts and toss them into a batch if it didn't automatically execute.
I don't think global needs to be specified here. The URL in globalAdminRestContext should do the trick.
Alright, let's use an exception then.
Yes, you're right... we should do this but it's not always easy to develop without modifying the formatting.
We should also clean the uncrustify conf and remove tabs...
should be string|null :)
That would make sense except my thinking is that this is an example of an extension that targets the DocumentView, not specifically the Editor object. There could be even more abstraction in places to develop an extension API, so that extensions don't have hard, breaking dependencies on MarkPad's internals.

BTW this is now in `MarkPad.MarkPadExtensions.SpellCheck.SpellCheckProvider, I did a spot of refactoring.
Неточность. Например 

* DNA  содержит TGAAA...
* indexes[i] = 3
* [cDNA objectAtIndex:arc4random()%4] = A

Фактически замены не произойдет.
Предположим, что выше Вы получили рандомных индекс, которого нет в массиве cashe. Доходите до этой строчки, сравниваете символы и получаете true, т. е.  символы равны и нам не подходят. После чего заново получаете новый рандомный индекс, который может уже находится в массиве cashe. Это не логично.
Чтобы оптимизировать алгоритм, вместо if используйте while и внутри тела цикла получайте рандомный сивол.
should indeed, thanks
Вот этим гемом следовало бы воспользоваться: https://github.com/jeffp/enumerated_attribute
Сделаешь, или мне его добавить, а ты спуллишь и воспользуешься?
Can this be renamed to BSONFilter, as ORM uses all-caps for the SQL acronym.
I'll make the move
We do this in a couple of places now.  It's a good candidate to become
a `getAboutURL` function or something.
Sounds good to me. There isn't a guideline, but there should be. The README is going to be the best place for that, I think.
Pluginen (altså `tabSlideOut.js`) har ikke hardkodet hvilke elementer den jobber med, så du står fritt til å endre som du vil. Det er kun eksempelet som bruker disse verdiene. Jeg ville endret elementet du kaller `.tabSlideOut(...)` på til `#feedback-content` og latt `.handle` stå som default.
I still don't like indenting private methods.
Just for my understanding: where could this method be used? In a class that extends ERXSingleButton?
请 参照英文版添加rev属性。
这些请求
=>
这些非法请求
Was meant to only be changed locally. I didn't realise it changed here. Will correct it.
Zooming out:

DefaultCamel has the same lifetime as the actor system it is associated with. 

There are a couple of internal actors used in akka-camel (mainly idempotentConsumerRegistry, activationTracker, producerWatcher), all started by start of DefaultCamel lifetime, but there is no overall supervisor for them. 

All of these are mainly for register/unregister state and notifications of ready state of register/unregister.

consumerRegistry and activationTracker have some state, producerWatcher has no state so survives restart automatically (but ProducerRegistry has state, which has lifetime of system and defaultcamel).  DefaultCamel shuts down on system termination, but does not stop the internal actors (which is not correct I think, they should be stopped, will add this). 

Since this is all so similar (and now in quite a thought through state) I'm going to take a crack at refactoring it to one generic solution.

IF for some reason outside of akka-camel, the 'internal actors' are restarted and camel and system keep on going, we lose information about registration of producers and consumers, and activation status (so listeners will timeout, and you could get double registrations). 

(I am assuming that now or in the future 'akka' can restart anything without warning, I have a dream where I suspend my akka cluster, 'serialize it' and switch it to another cloud provider and flip the switch ;-) but that's for another day)

If that is something that needs to be fixed, @viktorklang how do you recover state across pre/postStart/stop from within an actor?
Это урл на файл, просто теперь его излучает сигналом плагин.
ну могу палитрами задать
No need to call `.to_s` here.
Here is what you are, and who are there.
MigrationManager is missing from the pull req
@gscheibel if the entry already exists it should be replaced with the new one.

Checking for existance is tricky, as after you check it might be changed by a different thread. I would expect that MongoDB could provide you a method to insert the element overwriting if needed?
same here: -1 to instanceof usage
@dannygreg Can we add a comment to this code referencing that PR, just so we know that the cast is a temporary workaround?
Sorry about that... I was testing why pep8 (the program) was complaining on that line, and this is what it was asking me to do. I'm filing a bug report for that.
I think you lost some indentation here.
Delunay is a mpl sub-package, not the scipy delunay. However, there should be an absoute import of the package, i.e. ```import matplotlib.delunay``` rather than a relative one (can you check that for me?).
Please set the var names using camelCase
Imports on separate lines please
For what it's worth, we actually fire a notification when wifi comes up (mozWifiManager.onenabled). It would be good to fix this at some point.
if the socketnumber is smaller then 0 means that this protocol can be send to any socket. I wanted to add this to the documentation or should I add it some where?
Owww, yeah missed to delete that one. Used that one as template for the script, ohjeh - sluggish Saturday coding ^^
@davisagli: Yes, there are two conditions where we return early, although both are exceptional. I suppose the code could be refactored a little bit to do this little bit of extra work in the common case.
Remove the override for testing.  Also...I think using 0 for a valid user level is dangerous.  It evaluates to `false` and could cause some confusion later on down the road.
LGTM, but let's predicate macro this shiz in different pull request.
Seems unnecessary, as we only call this function when a widget actually needs to be lazy loaded.
haha #fail. So easy to overlook...
We need to use 1.8 compat syntax for the time being due to using JRuby in 1.8 mode
We hope to change to 1.9 mode soon though, but sadly for now you might want to use =>
"objects" is too vague to be meaningful -- I think it's primarily interesting to know how many file handles are open, given that there is a ceiling imposed by ulimit.  That probably belongs in the system statistics, then?
return (github = pkgInfo.repository.url.match(exp));
You missed this one too :P

... i'll fix her up.
You should add riak to the "suggest" key aswell, what do you think?
Can we make this condition `content != null` to handle both `null` and `undefined`?
Yeah, that sounds OK.
Unless I'm missing something, this wasn't supported before either. Think its a valuable addition, but not particularly tied to this feature either.
yes we can :). Will do and merge the PR if it's ok ?


On Sun, Sep 16, 2012 at 2:56 PM, Julian Descottes
<notifications@github.com>wrote:

> In js/service/LocalStorageService.js:
>
> > +            var target = $(evt.target);
> > +            if(target.hasClass("localstorage-restore")) {
> > +              this.restoreFromLocalStorage_();
> > +            }
> > +            else if (target.hasClass("localstorage-discard")) {
> > +              this.cleanLocalStorage_();
> > +            }
> > +            $.publish(Events.HIDE_NOTIFICATION);
> > +          }, this));
> > +        }, this)
> > +      }]);
> > +    }
> > +  };
> > +
> > +
> > +     // var frameSheet_;
>
> Can we remove all the commented out code ?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/juliandescottes/piskel/pull/88/files#r1614581>.
>
>
list($sortType, $sortDirection) = $sortParams;
@rorydriscoll, looks good save for this print. Could you remove it please?
You mean something like "unless which("nmap", ENV["PATH"]) " with no further tests? I can do that :)
Good! But order fields!
Exactly, will change it.
I was thinking about that. What about account-settings ?
But yes, if it makes things easier, you can try objectify.
Which other? I am talking about connections here. You can have multiple connections to the server at the same time. Each for one channel. So you have to keep track of them.

Look, it is very simple: if you know how to implement it your way, do it. If you do not, do it as I say.
What exactly do you want to achieve?
Why `/` instead of `\`???
Only first one.
Wrong indent (could also fix typo at the same time: their -> there
Leaving the timer in, but removing the dispatcher/checks, I can close, minimise, and hit save.
I would prefer these left on separate lines, as per our coding standard. (Favor clarity, not space reduction)
this will not compile - change these 3 lines
AuraEffectList const& meleeHasteAura;
meleeHasteAura = GetAuraEffectsByType(SPELL_AURA_MOD_MELEE_HASTE);
meleeHasteAura = GetAuraEffectsByType(SPELL_AURA_319);

to 
AuraEffectList const& meleeHasteAura = GetAuraEffectsByType(SPELL_AURA_MOD_MELEE_HASTE);
AuraEffectList const& meleeHasteAura2 = GetAuraEffectsByType(SPELL_AURA_319);
(while updating the for () loops)
This library is only compatible with PHP 5.4.
I think this description should be updated to include observations as well (since we're using it for both samples and observations).
you could mark it as required (the attribute used as key is the only place where it is possible as it is indeed required in each XML file, not in the merged config)
Read the man, your right, fixin
just testing that hasattr is True should do the trick. Oh, by the way, anyway I can reach you? Your mozilla email is dead.
i am not finding code for logout in browserid/pages...can you give me a pointer?
or i could just wait for is_logged_in to be false
// Convert `reason` to a rejected promise and use the general mechanisms of `resolve`.
Maybe move this to the beginning of `init()`, looks random here.
Inline function should have `{` on the previous line
That's fair.  The return body for inserting rows can be confusing, though.  The default is true.  I like the option of suppressing the return body.  I propose leaving the option; but defaulting to true.
Definitely.  I only did this because I was having trouble importing the project to forma-clj from Clojars with the SNAPSHOT version.  If you can figure out that issue -- which I'll open -- then we will immediately fix this problem.
Please just include the Core::ControllerHelpers module, as it will automatically include the other modules.
numpy indexing instead of for

cov_params[nz_idx[:,None], nz_idx] = H_restricted_inv
x[idx[:,None] * idx].reshape(idx.sum(), idx.sum(), order=...)
what determines the sequence in which this item show up in the table? and whether the info is on left or right?

Is it still possible to do 'Sample' in two lines ?
Shouldn't you invert the predicate?
Well, I read your comment about `units.hours-ago.1` and I understand the problem now. Your fix works for this case, but it'll break if one day someone will pass a hash with `'1'` key (as a string). But right here there's no much we can do because needle is just a string. In the long run, I think, a better solution will be to rewrite `KeyPath` so that it'll operate arrays of keys instead of strings where keys are joined with dots. That will allow us to distinguish numbers, strings and symbols because they'll be elements of an array representing the path and will have corresponding types. @camertron, what do you think?
I don't know, but I presume it's not stellar - I'm already looking into how to do that. First priority was functionality, since I'm less familiar with python..
Call the YozioHelper methods instead of Yozio methods (since you're testing YozioHelper, not Yozio)
List seems like the wrong semantic here.  Are they ordered?  Are repeats allowed?  Seems like Set is better (and aligns with getBusinessEntitySet API).
you will need a setter for this.
scale -> scales
This is clearly a bit of python magic.  Can you explain what is going on here? 

Actually, I think I probably understand most of this.  In pysrc/Image.cpp, `pyExportImage` sets up a dictionary called `_galsim.Image` which has keys equal to the python types (float32, float64, etc.) with values equal to the corresponding classes (ImageF, ImageD, etc.).  Then we assign `addNoise` as a method to each class in the list.  

But the kicker for me is that last line here.  Why `del Class`?
Seems like cut should minimize the width
```java
Math.min(width, img.getWidth());
```
that would make it safer.
I don't think you actually need to set these variables.  They should be set by default.
We need to thnik about where we will place our translations. In my last commit (which you can find in existing pull request) I've added translation file, but I'm not sure if this is correct way of doing this.
Try using `@Inject Help help;` to get this
It's a big performance hit, particularly if (as customers do) we are running with some debug logging. We end up calling AbstractEntity.toString a lot, and the groovy reflection for looking up fields reflectively is expensive.

The toStringFieldsToInclude is clever and concise, but I don't think it's worth supporting. Having multiple (incompatible) ways of building up the toString is extra effort and confusion, it's not a particularly natural/common API for Java developers, and it's a big performance hit.
I wondered about these demo.sh scripts. On balance, probably right thing to include these as it's the easiest possible thing for users trying out the system: it saves the user a few lines of shell commands. But it's a shame we need to include and maintain these files - at least they are massively simpler than they used to be...
Exists the same code under it.
I don't know the reason, there were a lot of similar problems.
Use `alias` instead of `super` just as I previously showed.
The preference that I set was for unvisited links, so I wanted to ensure that the link didn't somehow get clicked on in a different test which would then make it not unvisited. I suppose a simpler solution would be to set the same pref for both visited and unvisited links.
Should now be fixed. The problem was that older Ruby implementations don't directly let you access string character like `str[0]`, you get the character's ascii ordinal instead, so in the switch statement none of my checks for `@`, `+` or `-` were matching. With `str[0].chr`, this should work across all supported implementations.
No problemo, it's faster to implement that anyway.... new commit in a second...
You're totally right. There are a few other things we could clean up in the sphinx-specific view, but ... I figured we're going to ditch it in 3 weeks and so it wasn't worth making changes and then testing them.
Good catch !
What about linking the php.net bug in a comment ?
Oh I totally forgot about that. Will fix that soon
Looks like a line ending problem I had not so long ago. Are you coding under linux or you have autocrlf enabled in git options?
Yeah, it's also true you might find stuff like `[TrueClass, FalseClass]`, but I'm told that the official way is `[Boolean]`.

For singletons I would probably rather use the more common name rather than the class name.
Достаточно просто вызвать родительский конструктор, который принимает String.
Это некорректно. Копировать можно не только в директорию.
Получается, что ты на каждую сериализацию/десериализацию проходишься рефлекшеном? Не очень эффективное решение.

В конструкторе надо проверять, можно ли вообще сериализовать/десериализовать указанный тип. И готовить соответствующие структуры данных (один раз), а потом их использовать.
А этот код, похоже, лишний.
Нужно же как-то сделать так, чтобы сортировщики закончили работу в том же порядке, что и начали.
Based on your argument about storing multiple tokens, the URI is not the appropriate scope to pass in here, it should be the urn:mediaservices scope used to get the token in the first place.
I fixed this locally but then when I was testing `testswarm-browserstack` at the same time I couldn't get anything to work. Trying to make everything pass in under 10 seconds is very though if at all possible. I used "`browserSets[]=beta`" (only IE10) and running `testswarm-browserstack` manually from another console tab, and still it took 20 seconds total (run the script, a few seconds for browserstack to start it from the queue, then letting it run the test suite).

I'll settle for setting it to 1 minute in the test suite, and keeping your value of 10 minutes in the documentation.
We shouldn't tie the API instance to a single app, but pass it in for appropriate methods
what about cases where it is not defined ?
モック機能のサポートは、非ドメインクラスがドメインクラスを使用している場合に、データベースを必要とせず、テストに集中できるという意味では最適な方法です。しかし、ドメインの永続化のテストを行いたい場合はHibernateとデータベースを使用するIntegrationテストが最適な方法です。
にします。
@amueller Yoshua's book doesn't mention this AFAIK but this technique (replacing a sample with its expectation in a formula) is known as "Rao-Blackwellization", and it's trivial to demonstrate that it's still an unbiased estimator but with lower variance than the original sample-based estimate. See, for example, [this tutorial](http://www.cs.ubc.ca/~nando/papers/ita2010.pdf).
also should we add that these will be deprecated in version 0.14?
lower ? aren't supposed to fit first with high alpha?
@vene Actually I mean that those simple histogram-like calculation can be done using coo_matrix. Currently its performance is better than histogram2d. I will change this line :)
would it be useful to add a copy param to allow inplace squaring?
info = into ?
Même dans le cas des double? Ou alors tu parles seulement de la ligne mpz?
зачем? не используется вроде
Hmm that was weird. Removed and rebased it again.
write the test as a .sh test and create a dmd.conf and sc.ini config files with invalid include paths ought to work.  I could have sworn there was a command line parameter to specify the config file to use, which would have come in handy.  A bit tedious, I admit.
Variable e is not used.
Also we lost everything which is not .h or .a file, say common.mk, Makefiles get lost, 
I think we need to copy everything currently in that dirs, and then make build or anything we want to do on newly created items.
should have a s_ prefix for static
Consider using `[string stringByAppendingPathComponent:component];` here instead of a blind concatenation.
"a Lucene Query" is singular "which rely on " is plural.
correct, just pointing out that these changes should have their own issue & discussion. I don't see why you should keep changing one file at a time, especially if the change is the opposite to what we might decide to pick.
It is what would have been returned before.  (Both strings would have become blank and then compared).  Probaby more sensible would be to return a very large number.
Above, you have ```while (cur) cur = cur->next;```, but I don't see where next is set to NULL, to stop that loop.

Below, you just assign it to current attributes, which I couldn't find where they're initialised.

I believe attributes are only ever created here, so next will always point to an attribute, but I can't see where the first ever is set to NULL.
Do not afraid! Long names are fine! Use options instead opts.
alt text should be descriptive not a file name!
This image ends up being between the following text *and* the previous side note.
The text end up being really thin. Is there a way to force this image to be after the side note?
Repeating this patterns here seems like a bit of a repetition to me. Why not call the bundle above (rudder_common_report) using a methods/usebundle promise for each case?

Yeah, we can go meta on meta :+1:
Develop is being used as base branch for djangotoolbox and it has the bugged code.
@shacker Can we just use `@default` here?
That should also avoid the fact that we need to use 2 separate calls for this.
this trailing comma will break 2.6 backwards compatibility
@stefanv Sorry, I accidentally removed your comment! Hence I cite you here:

> If you convert the uints ``[0 100 200 255]`` to bool, I think you want ``[False False True True]``.

I'd expect a different behavior. But that is not a problem, if we clearly explain this in the documentation.
i have added a class level doc comment to mention MultilangRouteRepository
@anibalcucco noted. The app is pointing at the old URL right now, so will remember to observe payload changes when I change where the app is pointed.
Can this be generalized and moved to `CameraController`?  A `CameraTouchController` and some sort of camera navigation GUI would need inertia, too, so locking all this inertia and animation logic in this class seems wrong.
Why `@see Cartesian3`?  Do you mean `Quaternion`?
Needs `result` parameter.
Maybe we should reuse a constant here ?
Got it.  Yeah I agree generally.  Maybe Opie wants them in there for a little while as he continues to work on this web app, so it's "soft on the eyes" while he checks generated JS?  I think a happy medium here is to allow it as long as he has more plans for this app.  If he doesn't, then yank 'em.
You should register for the "postlayout" event on win, which fires as soon as the node has been layed out. This way the animation will start as soon as possible. Also, if the layout takes a really long time, for whatever reason, you don't have to worry about firing the animation too soon.
Looks like this file was checked in as part of another PR.
So this should be run from multiple nodes? Then the above is not going to be right?
Idea for title?
a) "jQuery UI {{component}} documentation"
There should be an argument which specifies what repository to add.
Can we do a check on both dispatchState and isDone? That way, you know that you are covering both of what other developers might do
does it make sense to assign the variable data here? looks like you could put ``$this.data('dropdown')`` straight into the if statement.
I apologize about this.  I didn't notice that Intellij did this.  

Why are build files included anyway?
You can think as much as you want but I have never ever any idea why such `rank` may be needed. And as the opposite, `staticRank` is needed for practical use in my other functions.
The intend here is that you can disable the SSSE3 implementation at compile time. This could be useful e.g. for kernel code or other restricted environments,
I do need it parameterized for testing; it says so in the code comment.  I can't override a final member in an anonymous class (not show stopping, but shitty).  This is a fool's errand -- honestly, overriding the method is the best way to do this.
Indeed.  The problem is that after the wipe, we need to re-calculate the set of folders written in the DB (which should be just places, unfiled, etc).  At the moment, this calculation is only done in `begin`.  I think it's legal to call `wipe` and then insert some things in the same session (which may already fail, depending on whether wipe preserves the special guids).  So I'm explicitly not handling this case yet.
I think this entire piece could actually be done inside of the JS.

I would create create the Liferay.Portlet.GoogleMaps as a widget that has a render method.

What you can do in the initializer is do the check if google exists and maps is an object, then call render, otherwise, you can use A.Get.script to load the js, and when it's ready, call render on the Liferay.Portlet.GoogleMaps instance.
@Provide
commas unnecessary
I think it would be easier to read if the following blocks are splitted into methods which are then called then once after the other
where do device id's come from? i am assuming `listDevices` but this could perhaps do with a comment here.
ah, you are learning. :P

Seriously though, nobody has even decided what we want to do for phpBB 4, nevermind how to code it. You'd be told to hold off. :P
because the for loop starts at -1, it will loop once. For instance, if someone picks the dates 8/9/12 and 8/9/12 and chooses to recur on Monday, no loans would be added, because the time span is for one day, which is a Thursday. Similarly, if the user chose the same dates and selected the loans to recur on Thursdays, one loan would be created for 8/9/12.
'goup' should be 'group'
yea, totally - second read of a clojure book is the best, stuff really
starts popping. I need to re-read it myself :-D

On Fri, Jul 6, 2012 at 5:01 PM, robzuber
<reply@reply.github.com>
wrote:
>>  (defn lookup-fn [namespaced-fn]
>> -  (let [[namespace fun] (split namespaced-fn #"/")]
>> -    (ns-resolve (symbol namespace) (symbol fun))))
>> +  (let [handler (:lookup-fn @config)]
>
> oh yeah, i just saw that in my book last night.  i feel like i need to re-read the whole thing now that i have a bit of experience.
>
> this lib was actually a great read in terms of understanding agents, state, watchers, etc.
>
> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/utahstreetlabs/resque-clojure/pull/1/files#r1116590
Rather than adding your repo at the top of the list, please follow the convention and insert it in alphabetical order. This will help keep this (frequently modified) file easily maintainable.
Same applies for other uses of `InstanceOf`, like

        $person = new Person();
        foreach ($contributor as $contributorInstance)
        {
            Validate::isInstanceOf($contributorInstance, $person, 'contributor'); 

In other places, the determination of type is based on the type name itself. For example, see the `Validation` class:

    public static function isDate($date)
    {
        if (gettype($date) != 'object' || get_class($date) != 'DateTime') {
            throw new InvalidArgumentTypeException('DateTime');
        }
    }

Perhaps that is preferable to using the **instanceof** operator.
regardless of use... the same routes would be generated (other than list, which would be index)
blocking call? not sure I'm a fan of that...
If this is some piece of desired functionality on the part of refinery, we might just want to `display: none` the title container, as lame as that is...
I added this parameter so users can run postgres on a separate server from puppetdb.  Without this, it would not be possible because postgres listens on localhost only.  They would also need to open the firewall port....
can you use `match` instead of `=~` or does that not work?
done http://screencast.com/t/TWPiKONj7d
context "given a single root node"
shortcut: `suite_importer or SuiteImporter(productversion.product)`
Regarding `!`, all the other operators have a space between them.  The only ones that don't are the increment/decrement operators; they actually change the value of the variable involved, so their behavior seems different enough to warrant a modified style.
2 spaces before `data-dropevent`
it seems you disabled the memorymapped file for win32 - so why do you include sys/mman.h ?
Common code from these parsers should be moved to HttpAdapterParsingUtils
Hello

Need " @since 2.2" notation
Gary, maybe will it be enough to add this operator to ```MockRestTemplate2``` and remove ```@SuppressWarnings("unused")``` ? ;-)
Or let it be: after all, this is just a test ...
Is this supposed to be adding back in "subscription"? That's what seems to be implied by the comment, but all this does is delete the "Subscription" property, not recreate a lowercase "subscription".
header comments should end with '.'.
this shouldn't be needed if the DataBoundConstructor is correct.
Nope...actually, it should not have been removed.  I blame auto merge foo.  Will add back.
Doug...you missed this :)
Interesting way to assure that `undefined` is not overridden. :-)
Same remark as previous.
type type typo.
I'll move jms/translation-bundle in "require-dev" because it is needed for the code coverage. However, the symfony/symfony dependency is something Rande wanted.
Same as above, mai bine ținem logica în cod Python.
I'm not a huge fan of names in the scraper - why do these have to be here again?
Please revert this file if there is no material change.
Sorry, bad find/replace on my part. Good Catch
kannst du die README und doku in eigene pull requests stecken? Übersichtshalber bei den issues
Priorities queues is only a partial sort and the underlying ADT is a heap (with a fixed size array). If you want a sorted data structure, use a treeset or a hashset.
Aligned!
Utilise before_save.
eek! - inline style?
it would be great if you could use "messageGuid" or something more specific than "guid" to help make what's happening more obvious (across all instances of this callback.)
tabs look a little weird here?
я бы добавил тест на измерение производительности. и мне интересен тест многопоточный, чтобы убедиться, что они будут нормально работать
My bad...not reading carefully.
This class should be deprecated, removed, and not exposed through the module API.
Security may not want us to log this to the console.
Wrong indentation here... you only need to indent +4, not +8 in the arguments
Good find. This throws off my idea of an swappable main column and sidebar. We should discuss this in the office next time we have a chance.
So how should we proceed in this case? I would obviously extend APE tests suite to include embedded containers where we need to register extension on the client side. Anything else I should consider here?
Do you have a script that can generate the tsv file?
do I need to start the agent every time?
The end goal here is to prevent id/parentId/folder from being changed. How should I implement it if we want to make it throw an error if people try to change it?
this must be flagged as an OSS issue.
Are we bundling this in the bar?

I need the following information for filing an OSS component-
1. Where we get this module from? link or website 
2. Version we are using?
`(not (teen? age))` välttäisi koodin kopioimista
The surrounding `if` is not necessary. `(== 0 (mod n divisor))` already has the right boolean value.
Unrelated to the pull request itself: Why not host them on the GitHub downloads page?
Please revert license header to original form, since the format change will break the Maven license check plugin.