labofsahil/github-event-dataset-2012 huggingface dataset:
I did this in a14a949f88ce6136407b4c6d4f9107609b861184
According to http://code.google.com/p/jsdoc-toolkit/wiki/TagReturns, a return tag should not have a name. We can remove `that` here and elsewhere.
.. + encodeURIComponent(contentId)
Looks like this is not being used in this file.
Being Java, you should know what exceptions is may throw...
Minor note: the overload helper for `query()` should remain just after the `query()` method itself, so I'll move this up.
"Допустимые" sounds a lot better in this context.
the spec says there's only one "ptr-var-sym {[ dim-sz ]}+" for each shape directive. why do you implement a vector of this kind ?
Ik zou consistent blijven en dit noemen:

`spottedLeft` en `spottedRight`
Curly brace needs to be on the next line, alone.

Also each occurence of `AND` must be replaced with `&&`.
А здесь мне, все таки не очень ясно. Суть моего не понимания: допустим, я переопределил метод init для объекта (наш случай Cell), после этого инициализация через init происходит только заданными в объекте значениями переменных. Как пользователь данного объекта инициализирует его другими значениями? Может быть в нашем случае это не актуально, но если это, допустим, необходимо.
Спасибо.
А какое преимущество? У самих указателей не будет меняться адресс в памяти?
В задании 100 )
Небольшой цикл, но я в нем не заметил увеличения count или уменьшения countEl, он у вас не зацикливается?
Both @derek-watson and I agree, I guess we had a miscommunication when we went over this on the phone. We will fix this.
sempre utilize var para declarar variaveis no js
This second sentence would probably be more appropriate in the "Returns" paragraph, since it's describing an attribute of the returned signal, not the "real" behaviors of the method itself.
Why the change in indexing? It seems unnecessarily confusing.
I changed this so that the it passed even though http://www.locomotiveapp.org should not be a valid variable name. I have logged an issue on the liquid git hub page about the VariableSignature regexp.
There is a silent change here: `getSM` would have checked that `mName` is actually a submodule in the owning repository. This doesn't happen here anymore. Do we need it?
Another non-critical note: `original.storage_object.master_copy` is a boolean and can thus never be `None`.
What's the purpose of this json?
ipython says %run plot_find_ecg_artifacts.py
although run works until you have a variable called run ...
2012/10/22 Martin Luessi <notifications@github.com>

> In examples/time_frequency/plot_tfr_topography.py:
>
> > +
> > +###############################################################################
> > +# Show topography of power (be patient, this may take some time)
> > +
> > +pl.figure()
> > +plot_topo_power(epochs, power, frequencies, layout, baseline=baseline,
> > +                mode=mode, decim=decim)
> > +title = 'Induced power - MNE sample data'
> > +pl.figtext(0.03, 0.9, title, color='w', fontsize=19)
> > +pl.show()
> > +
> > +
> > +###############################################################################
> > +# Show topography of phase_locking value (be patient, this may take some time)
> > +
> > +pl.figure()
>
> same here
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/mne-tools/mne-python/pull/87/files#r1904211>
> .
>

....common  practice -- but right, this one already returns a figure :-)

>
What are those two lines for?
Won't calling `compact` as the first method in this chain remove the first item in the array before it's used for anything?
只返回 -> 会发出 ？
默认不会编译
This is really weird. Can we do another type of check on ticketTags?
below you are reading `akka.actor.mailbox.amqp.*` from the resulting config, which would require the user to configure like this:

    my-dispatcher {
      mailbox-type = "...AMQP"
      akka.actor.mailbox.amqp {
        hostname = "..."
      }
    }

Instead I would recommend doing `userConfig.withFallback(system.settings.config.getConfig("akka.actor.mailbox.amqp"))` and then below `getString("hostname")`.
You should use ```get_by_key_name``` instead of ```get_or_insert``` - if that returns ```None``` we should return an error like ```error = "tag references snp which does not exist"```
Also,  do we always want to set a default language?
hm, I've missed that before. Keyword and variable names should contain only lower-case letters.
Is it necessarily an `NSTableCellView` nib? I imagine "sub nibs" could be used for other things in the future.
Added New Graph For Zooming Functionality.
Used a WeakkeyDict now. but not sure of the swapping the lines
If you're removing all tests from this class and you don't expect it to be executable, it'd be good to mark it abstract.
yep. this worked as longs as no virtual nodes were involved.
Because of the testConcurrency() method, it avoids duplicate entry errors
Do you have a use-case for this already?
It sounds like this (based on your documentation) that this is a relative path. If that's the case, can we rename it to `relativePath`?

Either way, we should also offer an `NSURL` property to this file, since string paths are sucky.
Multi-line docstrings should have the triple-quotes on their own lines.
Yeah -- we probably should use strings, but that would require changing all backends to be consistent and would require any third-party code using the event system to update.  Should be on a list of good "backward incompatible" changes to make some day.
should be ``` :class:`~matplotlib.patches.Rectangle` ```
It would fail fast if the atom could not lock the screen.
I see the `self.assertTrue` in other tests when unlocking. What are we asserting here? Can we just call lock() and leave the assertion out?
No, I want to do some testing between different apps like contacts and gallery, or gallery and Image Uploader with MozActivity. please reference [Bug 815059] (https://bugzilla.mozilla.org/show_bug.cgi?id=815059)
I usually try to scope all styles to the specific x-tag name.
self. аналогично
the problem is not the numbers but the missing {}

On Tue, Aug 7, 2012 at 11:09 AM, François Thillen
<notifications@github.com>wrote:

> In bridge/Implementation/OpenEngSB3_0_0/Remote/DomainReverseProxy.cs:
>
> > @@ -76,6 +76,7 @@ public DomainReverseProxy(T localDomainService, string host, string serviceId, s
> >          /// </summary>
> >          public override void CreateRemoteProxy()
> >          {
> > +            if (!registrationprocess.Equals(ERegistration.REGISTERED)) return;
>
> But how could I write this different, so that there is not only one if?
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/openengsb/loom-csharp/pull/23/files#r1319218>.
>
>
use constants?
Oh, you did -- then "avoid" is probably the wrong word, since you'll get an exception. start-index must be positive.
there is a merge conflict in scripts/tagger.rb with a bug fix on master branch. also, the right_link/spec/scripts/tagger_spec.rb tests have been enhanced on master so ensure your new code works with those specs.
Can we make the default log level `Logger::ERROR`?

The cucumber output is so pretty that way. Either that, or change your logger calls to `@log.debug()`
Couldn't it take the alt description from the image inside the noscript tag, so we don't need tp define the alt text twice?
Side-by-side is now disabled in IE<=7. We fall back to the "Preview" button that opens in new window.
Fixed in 3032ca8
oh lol, pretty sure thats copy/pasta from somewhere else (guessing scalaz). i don't use the unicode chars in type sigs usually
`unbind()` for sure... see the section on **Using Namespaces** in the [unbind documentation](http://api.jquery.com/unbind/).
Cool,  I'm not that familiar with types in redis.  You can replace the entire case statement with:

```
str == 'true'
```
I don't see the point of this code?  Why lock and unlock, with nothing in-between?
Lint says cursor->get_key can fail.

The function name in the comment is wrong (wt_metadata_get vs. wt_meta_btree_apply), this function doesn't call wt_metadata_read, so not sure about the comment, either.
This handler should be made available all the time now, not just during testing, since it's expected to take the place of the static mem only code.
Are you sure you want to override the parameter ``$modifiedField``? This often makes code less readable.
Please always use braces to surround the code block of ELSE statements.
Should the type for parameter ``$parent`` not be ``null|HasNodeInterface``?
I used the OneSwarm Timing Coordinator as a template for this file and only changed the handler so I will make sure to change these in the Timing Coordinator as well.
I might be missing some cleverness in `link()`, but what happens if the method has a sequenced functional parameter? Does it generate 'the right thing'?
this was simply copied and pasted from the KeyValueStoreTestCase. If there is an issue with it here, the Tests should probably be updated too.

https://github.com/doctrine/KeyValueStore/blob/master/tests/Doctrine/Tests/KeyValueStoreTestCase.php
ExpiresByType audio/ogg "access plus 1 month"
ExpiresByType image/gif "access plus 1 month"
  ExpiresByType image/jpeg "access plus 1 month"
  ExpiresByType image/png "access plus 1 month"
  ExpiresByType video/mp4 "access plus 1 month"
  ExpiresByType video/ogg "access plus 1 month"`
  ExpiresByType video/webm "access plus 1 month"
blz!
It could, but the only reason we're using this at all is because `Source` is an abstract base class.  The other models just use `upload_to="directory"`.
typo: Evaluations
Indent with spaces, not tabs.
Do it fast.

Please use `'` everywhere for constant strings.
```
which is set to request's GET `next` argument
```
I meant the commented out part. We should not have hardcoded values in our
code anywhere.


On Sat, Nov 17, 2012 at 2:06 PM, bkgomez <notifications@github.com> wrote:

> In fullhouse/templates/dashboard.html:
>
> > +					</p>
> > +					<p>
> > +						<label form="time"/>
> > +						Time: <input type="time" name="timedue" class="time-due"/>
> > +					<p>
> > +						<label form="assigned"/>
> > +						<p>Members: </p>
> > +						<select multiple="multiple" name="members" class="member-list">
> > +							<option value="1">Janette</option>
> > +							<option value="2">Conor</option>
> > +							<option value="3">Brandon</option>
> > +							<option value="4">Jeff</option>
> > +							<option value="5">Cornel</option>
> > +							<option value="6">Shahaf</option>
> > +							<option value="7">Nat</option>
> > +							<!-- TODO: Implement members, above are just placeholders
>
> It just makes the select list with the pre-populated names, the type of
> list that you can hold ctrl and select multiple items.
> … <#13b106859f400e7b_>
> On Sat, Nov 17, 2012 at 1:32 PM, Conor Reisman <notifications@github.com>wrote:
> In fullhouse/templates/dashboard.html: > + </p> > + <p> > + <label
> form="time"/> > + Time: <input type="time" name="timedue"
> class="time-due"/> > + <p> > + <label form="assigned"/> > + <p>Members:
> </p> > + <select multiple="multiple" name="members" class="member-list"> >
> + <option value="1">Janette</option> > + <option value="2">Conor</option> >
> + <option value="3">Brandon</option> > + <option value="4">Jeff</option> >
> + <option value="5">Cornel</option> > + <option value="6">Shahaf</option> >
> + <option value="7">Nat</option> > + <!-- TODO: Implement members, above
> are just placeholders What does this code do if run? — Reply to this email
> directly or view it on GitHub<
> https://github.com/Full-House-UW/fullhouse/pull/121/files#r2160285>.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Full-House-UW/fullhouse/pull/121/files#r2160336>.
>
>
I know that, I'd just whipped up a working version--there were still some
tweaks left, this is one of them :) Like I said, I'll split() afiilename by
slashes, and pass the array to os.path.join(). Or something like that. And
rename the variables so they make sense :)

Also, the script ensures that all folders aready exist, there's a
os.makedirs() somewhere above these lines.
ɹɐʞoɐƃǝɹoƃ ɥsıuɐɯ-


On Sat, Sep 15, 2012 at 8:21 AM, Savant Krishna <notifications@github.com>wrote:

> In main.py:
>
> > -							for link in self.br.links():
> > -								if (link.find("pluginfile.php")!=-1):
> > -									response=self.br.open(link)
> > -                	extension=response.geturl().rsplit('.',1)[1].split('?')[0]
> > -                	save_path=os.path.join(self.folderNames[course]+'/',afilename+'.'+extension)
> > -                	output=open(save_path,'w')
> > -                	output.write(response.read())
> > -                	output.close()
> > +                    response=self.br.open(allFiles[afile])
> > +                    if(self.getpdf):
> > +                        if (response.info()["Content-type"].split(";")[0]=="text/html"):
> > +                            for link in self.br.links():
> > +                                if (link.url.find("pluginfile.php")!=-1):
> > +                                    response=self.br.open(link.url)
> > +                    extension=response.geturl().rsplit('.',1)[1].split('?')[0]
> > +                    save_path=os.path.join(self.folderNames[course]+'/',afilename.replace("/","_")+'.'+extension.replace('/','_'))
>
> Ok. got that. Fine.
> But it's generally it's better to have the folder structure seperately in
> an array or something, leaving 'afilename' for just the filename, since if
> by chance a '/' creeps into the filename, there'll be an IOError as that
> folder was not created.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Manishearth/Moodle-Sync/pull/1/files#r1613230>.
>
>
Public API, needs NonNull check.
does this actually work? `_get_list` returns a generator, which won't have a `len()`, right? or am I missing something?
Why? They don't hurt anything and there aren't any official coding standards posted anywhere.
Kann ersetzt werden durch `startActivity(intent)`.
Holy moley. That's a lot of new Survivors.
This file should not be here. Did you rebase your branch from master before your updates? Your github master should always be in synch with my master. When starting a new development, you should: (1) update your master and (2) create a new branch with a descriptive name to implement your new feature.
needs updating to `radiusd -d /usr/local/pf/raddb/ -X`
>   What's necessitating `var x;`?

To fix the well-known scope problem.  I have had variables in functions
overwrite external vars many times and I use `var x;` to fix them.  Also
when I know I am reusing a var name, like `id` in many places I always do a
`var id;`.  My coding style has large files and I have poor short-term
memory.  I don't want to have to read the entire file above a function
every time I name a new var.

This problem is keeping a number of people from using CS.  Several rants
about the problem have made it near the top of hacker news.  Allowing the
var statement into CS would fix the problem, be backwards compatible, and
not bother people who code like Jeremy.

To say that many people who code CS have no problem is flawed logic.
There exist a better response for this situation than generic HttpResponse. Search for it in Django code and use it.
appear => appears.
taking => takes
Moved to an `IsDirective() function`.  No more magic.
I don't know. I just didn't want to touch Qbtsession. I just saw that everytime the user makes changes in the options/preferences you call this method to reconfigure the session. In our case the user changes the options via the "Set as default" combobox. In configureSession() the preferences are opened and read again(that's why the .sync()) and defaultSavePath is reassigned. I am not sure what else we could do here.

We could make the preferences class a singleton (and maybe wrap the pointer in a smart pointer class) and make every class that needs the preferences have a member value that holds this pointer. And then use the pointer to get the values directly and not store them in member values (like above). This will have the advantage that we won't have to worry on how/when to update the relevant classes. The changes take effect immediately application wide. The disadvantage is that a lot of code would need to be touched.
@sorin-ionescu Chakra [has](http://chakra-linux.org/ccr/). I've never heard of any of the other distributions, and Wikipedia didn't help.  I doubt that they have any users other than their own creators at all...

Well, not having the latest stuff is a problem if you want to test a new tool or port your program to a new library, that's still lingering in testing. Developers of pacman frontend couldn't configure their system to get just pacman 4 and nothing else from testing. Either they had to use the whole of testing, including other bleeding edge stuff like "kmod" which was in testing at the same time, or they had to carefully disable "testing" before each `pacman -Syu`, and upgrade pacman separately afterwards with "testing" enabled again, for once you've "testing" packages in, you can't get them out again easily, because pacman can't downgrade all packages from a certain repository.

Debian has debhelper as packaging tool, and not much else. Standard stuff like CMake builds, Python packages or Ruby gems are easy to package, though it's admittedly more work than PKGBUILDs. Btw, Debian also has pbuilder to help you get the package dependencies right and avoid automagic dependencies. Arch doesn't have anything similar, and suffered breakage in the past, even for core packages, because packages silently build against optional libraries that weren't declared in the dependencies of a PKGBUILD.

These are really dead simple to create, but also insanely hard to parse *securely*, because they are bash, and the whole of it. Folks use all too clever(TM) expansion tricks and conditional stuff, and split packages are based on bash arrays. If you can't use bash to parse them, you're screwed.The AUR [still doesn't support split packages](https://bugs.archlinux.org/task/16394) and shows [broken URLs](http://aur.archlinux.org/packages.php?ID=51695) for many packages!
this had installation.html at one point during rebase, but it got removed.
Where are the install instructions?
Is there a reason we need to save this right away? Wouldn't saving at shutdown be acceptable? File IO is terribly slow.
OK, pushed, all tests pass.  Caught one more bug this way too (also in Airy, a missing pair of parentheses in a call to `SBProfile.getObscuration()`).

I think we may be nearly there.  Thanks for these comments Mike, made a vast difference.
should have a line checking that dev is a BaseDeviate before using it to make a GaussianDeviate?
Ah yes, but it also mean that the fix to be complete should also change OpenCOLLADA to rely on the version produced by this config file instead of its own define (need to check if that exists). I'll ask NetAllied folks about this.
What is this???
if GC is ever that big it needs to be sorted out
is it really this simple?  does this work against your nimbula image?

also -- are these values always set?  could you check out #332 and add an OS check to one of the tests which uses us-east-1/ami-f95cf390.  (that's a good image to know about because it is configured with ec2-user but jclouds isn't familiar with its naming conventions.)  i was thinking we might need to ssh and uname on the box to discover these things reliably... ?
Is what you want `Class.getEnumConstants()`, rather than doing reflection here?
I was suggesting that the methods (e.g. create) taking a template as an argument continue to take that template argument.
But the methods for registering templates etc don't seem to do anything except subsequently affect a couple of log statements. If we want some sort of template registry, then could that behaviour be extracted? Am I missing something?
And ditto in other test methods.
La je mettrais juste un exemple pour illuster le less privacy
s/to/by/
This tested Stanza.submit passed an already encoded string. We removed the support for strings, so the test goes as well.
Might be useful to restore the comment:
`# Wait until the event count goes to zero`
Just made an update to handle this and merge with 0.6.1
This is more a utility module. Basically, it just checks, modifies and splits path strings (like node.js's path module). I do use the .webinos directory. 

On 10 Oct 2012, at 16:28, Habib Virji <notifications@github.com> wrote:

> In webinos/api/file/lib/virtual-path.js:
> 
> > +    return root + dir
> > +  }
> > +
> > +  exports.basename = function (path, ext) {
> > +    var file = splitPath(path)[2]
> > +
> > +    if (ext && file.substr(-1 * ext.length) === ext) {
> > +      file = file.substr(0, file.length - ext.length)
> > +    }
> > +
> > +    return file
> > +  }
> > +
> > +  exports.extname = function (path) {
> > +    return splitPath(path)[3]
> > +  }
> are you planning not to use .webinos then what will be the basepath used by file api. any current directory
> 
> —
> Reply to this email directly or view it on GitHub.
>
Please remove these comments. They just make the code looks ugly.
Yes, that'd be better.
I erm
deleted the main map when I renamed mine!
We can probably have some tests for this class in isolation of the plugin functionality. That might give us some opportunity to refactor this so that we have some better MVC in here.
I'd put a Q_ASSERT here - this should _not_ happen (and Q_ASSERT disappears in release compiles)
Same probem here, a solutions which solves both at once would be nice.
Actually I should be able to use the applyPid method now.  Good point.
Same as above, this should be ApplicationExportManager_export
Looks like Data.java belongs to datamanager package. Am I wrong?
Isn't it already somewhere? I think we could place it somewhere in lib and include where needed.
Мне это сделать или пока и так сойдет?
Об этом я и говорю - но мне удобнее проверять есть ли не следующий, а текущий. Могу переписать, но по моему сейчас все выглядит более менее лаконично и логично.
You're not way out in left field on that...  Python's SSL implementation has the ability to verify the server certificate against a CA.  However, the *requests* module doesn't appear to support it.  At least, I haven't spent enough time digging into it to figure out how do to it :)
I posted this in January on StackOverflow, called it getExtendedAccessToken, seemed most logical to me back then.

You can see it here, http://stackoverflow.com/questions/8982025/how-to-extend-access-token-validity-since-offline-access-deprecation/9035036#9035036
Any thoughts if this check will be compatible with the ADHOC wifi mode that _hc's OLSRd app puts the Wifi in?

I am tihnking there might be an issue here.
Cool, thanks for documenting
yes the pages are the same we use https://developer.paypal.com for our pay pall tests so we could create a sub-module for this pages
you mean like that ? :)
Yeah. I had it in but removed it because I didn't use it yet...
why `:input` is getting a different approach?
I thought a rewrite would be too much to ask, but that's great!
> If this goes into fit, suppose user uses
> the private function without fit for the first time, the function will fail as
> we may use self.random_state directly in the _spectral_embedding.

It's a private function, so all bets are off. For all we know, it may
create a black hole if not used properly :).

I wouldn't try to design for the use of private functions, so I wouldn't
worry about this.
Could give some more details here?
can you add a score method being kendall-tau?
can you tweak bit the figsize to the text shows up well?
You're right. Whoohoo, less code! ;)
а у нас есть поле рейтинга :)?
doesn't checkstyle check for redundant modifiers on interfaces? At least i remember complaining about that at one point, haven't double checked though.
Oh, and you can set the `1.6.*` version, or `1.6.5` which is the last tagged version
The docs say this is only for iOS
It's a matter of taste and i can see benefits to it.  I might do that as in part 2, but i don't think it's an issue that should prevent the patch from being approved as it bears no change to the actual functionality.
`we can' instead of `when can'
For consistency and pleasure of reading the SDK functions
I don't think standAloneDll mode should imply dynamicFileResolution.
Is setting to NULL necessary?
same below...
does nova on vmware still require the libvirt service?
You know what i think about the local cache ;)
This is the type mistake,
the Origin line is "if model =~ /IBM/"
just miss the space
hmm reading that comment again, it might look offensive, dont mean it like that at all, i really am thankfull :)
Same height for the borders
caught nil errors while fiddling with the database during testing. Wrote up spec tests that will cover this.
shouldn't the null here be "" ?
This is wrong. getParent is used **before** buildForm
Please, use the same format than the original module for the "print_*" messages, prepending something like:

"#{target_url} - WordPress Users Enumeration - blah blah"
Forgive me I am unclear on your comment.  Are you saying that I should not return if the exception occurs or I should return something like perhaps output_error maybe?
Agreed, included the same.
Yes, I'll do this later today or tomorrow.
Should be `List.foldr` :)
We don't use this approach in the quickstarts, so can you remove it?
这个个写法好
I've been treating the atlas like a square. Should I change it to work with rectangle textures?
what about the concice form `background: url(../Images/ajax-loader.gif) center no-repeat;`
I'll add that my general feeling is that CameraController should be a best effort class that works across all scene modes.  Having certain functions only work in certain scenes makes for a confusing API.
Think about removing duplication around creation of cmdlet - use a factory function?
Should be "Reads data from the stream..." I think
autorelease.
We should try and avoid a more generic base class knowing about specialized children.  We should ensure that we maintain solid encapsulation and inheritance model.  If need be, we should implement a TiUiAlertDialog that extends TiUiDialog the same way that TiUIImageView extends TiUiView in order to support the isPersistent property.
Here you're specifying ".tgz" as the file extension.  That implies gzip, but you're not specifying a "z" in the tar command (so no compression is occurring -- you're just tar'ing but not compressing, though you give it a name that implies compression.)
For header and footer views we should be using `rememberProxy` and `forgetProxy` to ensure retention.
Get rid of the word "and".
all imports should be with a star, eg:

`import java.util.*`
Sí. Incluso se puede utilizar una lista, para evitar el espacio generado entre enlaces. (ej.)

* Twitter
* Facebook
This is a generated test file. The comments should only be made on files under `test/` or `lib/`. Stuff under `client/` is autogenerated.
Тут точно неправильно. :laughing:
Yes, you have already fix it. Sorry!
Please change this to _category_item_locator so it's clear that it's just for one item instead of more than one (implied with usage of items)
It may not be as performant, however you'll probably get better support for various browsers with `$.getScript()`.
I think the app server will just use it's own persistent store for that, not one you define.
Can't you do with something along the lines of

```
bindings[key] = bindings[key].filter(function (x) x !== binding);
if (!bindings[key].length)
  delete bindings[key];
```

?

Also, is it an invariant of your code that any array that is in `bindings` is non-empty (i.e. that you don't allow zero-length arrays)?
Why not just ```[^'"]*```?
oops, sorry for that :)
There is a bug with this option #301 and %locale% can not work in all cases
No, E-mail has no translation in Bulgarian, it's used as it is :)
why are there 2 listeners? maybe do this in 1 class?
This requires the ```presenter``` to hand you yet another sub presenter, which I'm not a fan of, as it makes it unnecessarily complicated on the Woople side, and feels like a pre-optimization for the electives section.

This would be easier for the Woople side, as it only requires the Woople side to only worry about handing over a presenter that answers questions we need, and we then wrap it with our own presenter on this side of town for any other questions our view on this side of town that requires extra logic:
```
def essentials_section(presenter)
  render partial: 'dashboard/essentials_section', object: ThemePresentation.wrap(presenter, WoopleTheme::Dashboard::EssentialsSectionPresenter)
end
```
The trailing tag should be `</p>` instead of `<p>`.

Other than that it looks good to me.
"Travis Pro" should link to http://travis-ci.com.
We don't, good catch.
Let's leave it like that so we don't break client code but I just added a [bug report](http://d.puremagic.com/issues/show_bug.cgi?id=8132) for this.
You're not actually calling anything that requires the service, so you don't have to wait for the proper wait to mock to test this.
Ditto, magic constant
This line must have the same length as the title
This if block can be removed because the driver will throw an exception before it gets to it.
Hah, I should probably update this seeing as I wrote it and clearly this isn't internal anymore (I had it as a gist I sent to a few people, not sure how you got a hold of it :-)
You should probably only use translate3d for WebKit right now. Use a regular translate(x, y) for the other vendors and remove perspective for the non-3D vendors.
The values of `seq1` and `seq2` never change, so this only every compares the first elements of the sequences. No points from this one, but you can try to fix it.
I would rather not impose the restriction that backing up globals be configured off. There is no need for the $moduleManager to be in global scope, I think it's better to unset here.
Not sure what you mean....I use this setting on the Reply To server now so that I can see the Mandrill API responses.  If you run the tests, you do not see these lines because the default is to not output them.
this should also be a <many-to-one>, just like sysUser
replace this javadoc with something useful (like below).
Copyright Cesium Contributors.
Weird, did you actually see how that happen?
I think I would just get rid of this.  In the one place you're using it, you could just as easily swap while assigning the private fields.
同样问题：这里你不需要碰，就别碰
Répétition de autre
Shared scripts is a feature that is coming. Once it is in, this function will encapsulate the logic to set/retrieve the shared scripts without requiring changes in upstream code.
You are correct, it should be removed here. i will remove it.
Ah, now I get the confusion. ```unlockProperties``` will revert the lock to NONE on ```retry```, not back to read-lock. This is a temporary condition, only until you commit or rollback, in which case, it does revert to read lock.

That was the whole point of having commit and rollback as separate boolean arguments, remember?
you should keep ``Doctrine\\ORM`` IMO as it is the only one which can be found there
In android, it can take 2 more optional parameters: locale and pattern.
This still isn't a property. Set it as a property.
change the name of this method to "isAlreadyBeingProcessed" or something a bit more informative.
(it's unclear what it is doing down when it is used)
I squirm a little bit every time I have to use old-school array indexing like this, and there's an assumption that every caller will have columns in the same place, but I also try to avoid duplicate db fetches. Does this work out to three fetches (rows, count, cost), or does AR optimize part of it away via caching?

Yeah, I'm being pedantic. Habit of a former DBA with a well-earned mistrust of db abstraction. For this app you won't be summing across millions of rows, and your suggestion is much easier to read. But I would still like to understand how it works. :)
The figure is quite large.
Maybe use fewer examples and do not pack too many possible scenarios in one figure.
Do you need to wait for the SendAsync to complete?
O.k. :) I will try with TomEE for APE then.
But we need some clarification from @aslakknutsen
Remove the maven/Npanday files. When you can't use it, it may confude other user. Add the dlls without the Npanday
Here's another example of why this entire file has been marked, again.