labofsahil/github-event-dataset-2012 huggingface dataset:
Not good. What it there is not e-mail information? You should add to `if` above test if we really get some username value out of it.

```
if user.lazyuser_username and foursquare_profile_data.get('contact', {}).get('email', '').rsplit('@', 1)[0]:
```

Of course you could store this into some variable before if. Like `username_guess`.
*Prijava uspešna.*
No no. Not here. By something else, some other aspect of PiplMesh.
It makes the appropriate `browserid_form` variable available in `RequestContext`. Not sure if we need it at the moment, but it might ease handling of `RequestContext` in the future, if we need to extend it with anything.
Do we have other translatable string in Python marked as Unicode with `u`?
Spell out the variable name fully.
`;` at the end.

Nice. :-) Is there a chance of infinite loop?
This is a dangerous form of importing - it works as a ''relative'' import (because there is a module called "attachment" in the same folder) but if somebody else was to put a module of the same name on the path (for instance, in the standard library), it might break.

What you really want to do (and this is why PyDev does this per default) is `from tractor import attachment`; however, that does not work because you chose to have a module named "tractor" in the same folder.

For now, we could make the import safe by changing it to "from . import attachment" (this is an ''explicit'' relative import which will always look inside the tractor folder), but I propose to get rid of the `tractor` module down the road.
I don't think this is safe, as it would force a switch to LLVM on systems like Leopard and Snow Leopard where LLVM still has significant problems.
Remove this case...
These imports must stay inline for now... There are some developers who can not install numpy yet and it causes generate interfaces to fail.
probably shouldn't be a map to start off, maybe a fxn that does nothing?
Nice, thanks. That's definitely an improvement. Tweaked that, along with a few other changes (DRYer code, `brew audit` included).
Since Substitute's `For` method is a static method, perhaps this class is not needed. (See the comment below.)
this change is wrong. ``@`` at the beginning of a value is reserved for future use in the YAML spec, so you should keep it quoted
What it is the reason for it? o_O
aura script is indeed better.
Yeah, that's why I went with for Common Salvage at random. Partly to be different and partly to be less obnoxious. I'll add in 'the most salvage' bit and see how it goes. Could be funny if the person with the largest hand has only Rare.
Here is a simple cheat sheet to get you started with this type of setup
Not sure if i'm following correctly, but for each defined SNAT interface, if you have a defined network that will use routed rather than nat, both rules will be created ?
It's not relative to the document, it's relative to the screen. I think you can just delete the ", relative to the document".
The output print stream used by the kiji command line tools is configurable. You should use the getPrintStream() method to output, not System.out.
Please add a TODO to refactor this to sahre as much as possible with apache_cache..cc.
you mean it must be in lowercase. Ummm, yes sorry... :)
El 29/10/2012 14:27, "Brian Ronald" <notifications@github.com> escribió:
>
> In data/lang/Spanish.txt:
>
> > @@ -484,8 +486,12 @@ HYPERDRIVE
> >      Motor hiperespacial
> >  FREE
> >      Libre
> > +FREE_LOWERCASE
> > +    Libre
>
> not "libre"?
>
> —
> Reply to this email directly or view it on GitHub.
Went ahead and added a transform keyword and some docs. Looks like we might have to do the doc templating by hand for each class?
I think it'd look a bit nicer if you extract numeric constants from this method into a hash with time unit symbols as keys. Then you can rewrite this method into smth like

```ruby
def calculate_time(seconds, unit)
    round_to(seconds / TIME_IN_SECONDS[unit], 0)
end
```

What's the purpose of `round_to` method in this class if you always pass 0 into it? If you're going to use it with a different precision in the future, I think we should extract this method into `Twttier::Utils` module, because exactly the same method exists in `NumberFormatter`. Using the same method from both classes is much better than having its local copy in every one of them.
Leave copyright to wlan slovenija. Project will be deployed there and maintained by the community there.
png.gif
Obscuration??
Perhaps `(i.e. a Sum or Convolution)`, since there are only two it doesn't hurt to be explicit!
You have taught me well, Obi Wan.
might we want to `.cue()` all of the slide indices as well?
Or I can. Thanks for catching.
testing 107 r
Now uses BrooklynSystemProperties.DoubleSystemProperty.
Commit rebased and done `push -f`
I'd do `checkNotNull(is, "is")` at start of method, before creating the temporary file.
What's the difference between "attributes" and "fields", as separated by the comments? They all have getters and no setters, and all exposed on the CatalogItem interface.
I'd probably do the if.isDebugEnabled block inside the synchronized. Otherwise, different threads can interleave their log statements and it could be hard/impossible to piece together from the log file what the actual state of the object is. If log level > debug then should have almost no impact...

That involves trusting no-one will override the methods it calls with deadlock-prone code...
Can you make this the same before, one line, no variable, just return and indent to clarify.
could we do something like create a profile that sets a preference such as `browser.startup.homepage`, then use that profile and verify the initial URL?
This one, with an array of 300000 and chunks of 13 took 22ms.
It returns functions as it is checking `typof obj == 'function'` where in ES5 `'function'` means `Object (native or host and does implement [[Call]])` (http://es5.github.com/#x11.4.3).

I can't think of a case (and there is not one in the underscore test suite) where `typeof == 'function'` would return unexpected results, or results differing from what `toString.call == [object Function]` would return.
Agreed. Not going to happen right now. skip_refresh is there only to work around changes in how Job is handled.
це перегляд категорії, тому давай це в контролер категорій
урл відповідно зміни
Could at least keys be `10s`? Do we use them anywhere as JavaScript or Python object attribute value? So `object.10s` or something?
Can you delete this file in your pull request.
Can\'t
you can put the call to setCheckable() before the call to super(...) with a comment indicating why. 

However, it may be cleaner the way you have it proposed. In that case, there should still be a comment explaining why it's being set there versus in the initialize method.
I wondered about how to implement a Predicate that used the other Predicate.  Very cool!  Thanks
I don't agree. More common != more readable.

Also, I think that `version_compare()` is more error-prone, see: 515b5819374a1a207944c46ff9876ee607cc965d. 

Using `PHP_VERSION_ID` you won't make this mistake since it doesn't include extra version.
Might be worthwhile as a minor tweak to move this onto the WHERE line above just for symmetry with the below two examples that are line broken differently.
3 of [anything] is probably the time to switch to a List of [anything]s.
80 columns rule?
Make sure wherever that is reverenced that it is mutations2, not mutations.
e.g. if(mutations2 & NOCLONE)
This actually isn't considered a public interface (there is no injection point in the framework to inject your own implementation...that's what this change creates.  Dave's comment in the Jira issue:

"Since the existing interface is internal (no injection point)"
Yep, I just add commit which fix that
Поясни, что значит это исправление.
Beware of  ```removeObserver:forKeyPath:context``` method, it doesn't support on iOS4.
This kind of per-function comment is unnecessary. When I see comments like this, I'm always skeptical that they will stay up to date as the code changes with time. If your code is factored properly, and you've named your functions clearly enough, these comments are just extra noise and a regression risk. This one is a great example, where you've got a function called "initVideoPlayer" which, not surprisingly, "creates a videoplayer."

Use comments when you need to communicate a message, not to add redundant information to your function definitions.
these are not new dependencies. On my machine, without these lines, it just won't build. I think surefire is required by our code coverage tool, isn't it?
Why is `force_unicode()` needed here?
I think I would rather have a function ``deterministic_svd`` or just a function ``svd`` with ``make_deterministic=True`` or something like that.
We could even check the input in the wrapper and yield a meaningful error message if the second input array has a 1D shape.
Let's include uglify as a submodule.
Путь простраивается от типа запроса. 

У нас  есть запросы вроде /files/%file_id%/storage. Заставлять разработчика прописывать такой путь - не корректно, по моему мнению. 

Все внутренности методов и работа с REST в рамках реализации должны быть черным ящиком, больше чем дает документация разработчик знать, по идее, не должен.

Это не работа с REST, это определенный тип запроса к определенному сервису.

Отсюда и набор констант - это моя забота, как разработчика библиотеки задать определенные константы, чтобы разработчик использующий эту либу работал в рамках.

По поводу:

    $api->request('GET', '/account/', $data, $headers=array());

GET - это необязательный параметр, изначально по дефолту всегда делает GET, а вот тип запроса нужно указывать, в итоге он первый в списке параметров метода.

$headers - лишнее, заголовки должны быть только моей головной болью.
This was an accidental commit...
FYI, there's a JIRA to fix our logging of IPv6 addresses to comply with some RFCs (https://issues.jboss.org/browse/AS7-3847). I think to fix that we'll need to add a method or two to NetworkUtils. And probably a class in the same package that can wrap an InetSocketAddress and return an RFC compliant string for the host:port part of a URL (e.g. [::1]:9990). Once that is done, an instance of that class is what we'd pass to this logger method.
s/Manhout/Mahout/
On Sat, Sep 29, 2012 at 01:39:48PM -0700, Hara Kenji wrote:
> This is redundant code, because `getDsymbol` catches `TOKvar`.

ok, removed. The DotVar one is still needed to pass the test using
traits(getMember) though.
It means quietly semi-ignoring a bug in dmd (if Terror ever got here).  Wouldn't it be better to fail loudly and quickly?  I think it should really be replaced with an assert.

I also think all of the `default: return "unknown";` cases should be replaced with asserts.  This switch is missing a default too.
`error` prepends the module name, `::error` does not. Otherwise the error message would end up being:

```
Error: module object cannot find source code for runtime library file 'object.d'
```
I don't know this part of the code, but why is this conditional on being a property?  Feels wrong.  Also feels like if this one should be then so should the one in the true branch above.   Assuming they should be the same, the whole suffix handling should be after the conditional.
unusual indent - vertical align doesn't quite justify it
keep odered
Ah oui, c'est vrai que tu as un Form[DocEtude] et je ne sais pas si tu peux appeler les fonctions de l'objets à travers le formulaire... à voir
Si on ne peut pas, tu as deux choix :
- des fonctions statiques dans DocEtude
- on passe également l'objet à la page.

Perso je sais pas trop ce qui est le mieux... (léger avantage tout de même pour la deuxième méthode (meilleur respect du MVC))
Sorry, that's true. I'm not sure how that ended up like that :)

On Oct 8, 2012, at 10:57 AM, Kyle Robinson Young <notifications@github.com<mailto:notifications@github.com>> wrote:


In tasks/sass.js:

> @@ -42,7 +42,7 @@ module.exports = function(grunt) {
>        });
>
>        var
> -        isWin = process.platform.indexOf('win') !== -1,
> +        isWin = process.platform.indexOf('win') == "win32",


Shouldn't it be isWin = process.platform === 'win32' then?

—
Reply to this email directly or view it on GitHub<https://github.com/gruntjs/grunt-contrib-sass/pull/3/files#r1787229>.
I guess this one (but not the next one) could be a break (and thus be able to use SOME of the following 32 pages even though all 32 are not available.
well spotted.
We should be explicit about where the programs will look for the ecl.ini file (is it local directory only? Should it be?)
+1 on that. Just in views.py where it's needed. Then if you find that it's repeating itself, then it can move to utils.
I guess not, unless there is some concern something else would try and has counter_type/counter_name. That's why I didn't change things.
I have a non-standard install, (It's in `~/.homebrew/`) and it works just fine for me, because homebrew automagically puts a `-I$(brew --prefix)/include` into `CPPFLAGS`.
The else clause handles the block at line 6, <code>before { subject.visit("/hello/world?success=true") }</code>
```C
int main(int argc, char *argv[])
```
hould this be here?
@shacker Can we rename `col` to `column`?
Kuten myös `(and (<= x1 a1 x2) (<= y1 a2 y2))`.
Need to make sure Uri is encoded, use Utility.SafeEncode
If Tod was reviewing this, he probably would have asked you to randomize the data part instead of 'test'...  it doesn't have to be 'test', right?
Buffer overflow in the CxDbgPrint function in the ipswcom.dll....
"Sigma" -> "kernel_size"?
The checkbox need to have the id `toggle-all` to make the label clickable.
I agree about the to_s, but I’d like to keep the instance variable around. Initially, I wanted to use Kerberos authentication, which is possible with Curl, but you need to set some properties on the instance variable. However, currently, I need to use SSH forwarding because, for some reason, Kerberos + HTTPS is far too slow.

I changed the code to use to_s.
Seconds is always an integer by this point, so my check is correct, however I think >= 61 is clearer in the code so I'll change it.
If you remove this bit, I can merge the commit above. Else I'll wait on #12 to be reviewed by @frodenas first.
You could improve the comment with examples of what is the correct version format
consistence would be nicer here, I'd go with public ...... for everything.... or nothing
To maintain consistency an clarity, this should be changed to EVENT_KEY_PRESSED as EVENT_KEY is fairly generic and can lead to confusion in the future as more events are added.
whats wrong with explicitly checking the value of a boolean? or is there?
Definitely. For Macbook Pros (vintage 2010) `en0` is wired, `en1` is wifi (at least on models with both) and there may be `fw` or whatever Thunderbolt is on newer models which have an ethernet adapter attached.
some comments about the purpose of these files would be nice. took me once ages to find out about jboss-deployment-structure.xml. Also faces config etc. Generally we should describe what we are building and why
Although you noted that these are put in the global window on purpose, I have to insist that it is not.  Why not just create a separate namespace?  The ideas is to have maximal global abatement. 

We can call it `Utils`, as in `localRedis.Utils`.  In fact, it would be in good practice to create a namespace for any future functionality not in the API extension to localStore. What do  you think?

ie.
```javascript

if (!localRedis) {
  var  localRedis = {};
}

if(!localRedis.Utils) {
  localRedis.Utils = {};
}

(function (utils) {
  utils.tick = function () {};
  ...
} (localRedis.Utils)); 
```
This way, we can have say one file to hold libs.  Say local.redis.libs.js
It's a typo. Should be @disabled .
Оукей
Я уже ни в чём не уверен. `return false`, по идее, то же самое, что `preventDefault`, но с последним происходил редирект при дропе. Могу поэкспериментировать ещё немножко.
давай заново учить функции из algorithm
> I also don't agree with @phiggins point that it's not a good idea to inherit from ruby basic types.

MRI cheats in its implementation of core classes. Sometimes, you won't get what you expect. For example, sometimes, when creating a new string, #initialize does not get called.

Unless you're willing to memorize these exceptions, best just to compose with core classes. Duck Typing basically means that inheritance is almost never needed.
I think we should use `__traits(compiles, ...)` for new code; it's less confusing.
I think raher than go into /etc/cron.hourly, which is run by root, it would be better to just edit the crontab for this user.  One advantage is that that way you can use MAILTO rather than having to mail explicitly -- it will automatically do the thing of only sending mail if there's output.  Another advantage is you don't have to guess where $HOME is, you can just use $HOME.
I Inlined it instead. I used to call it from other places, but now it was only invoked here.
yes, i think i was testing something and didn't revert the change to detect_sbt.
mhm.... WDYT about introducing a filter builder? I think of something similar to criteria? FilterBuilder.and(FilterBuilder.equal(CONST1, x), FilterBuilder.equal(CONST2, y)).build(); Maybe in a separate issue?
Why using MyImprintPanel class and not ImprintPanel interface? Is there a specific reason? If there is, sry I haven't seen it. If possible, you should always use the interface instead of the real class : ) Compare it with:
List<String> test = new ArrayList<String>();
and
ArrayList<String> test = new ArrayList<String>();
Ah, hmm.. Well, you should have all the "rackified" headers directly in the ENV hash (as per Rack spec). I guess if you wanted the raw headers, then that's a bit different.. /cc @dj2
Is this something we can try and merge into https://github.com/benmatselby/sublime-phpcs. The code looks to follow the same structure, so adding this in shouldn't be too difficult. More than happy to do this if you don't mind?
180339f
So passing empty array is accepted? Just curious.
Yes this will ease the composing of the Authentication Scheme in the `ServicesBuilder` so users can just pass their own authentication scheme and the filter use that directly.
Why have two separate easyconfigs?

It's better to only have one, imho, without a versionsuffix but with extensions. We don't add version suffixes for Python either...
Yes it must be in the trace level. The warn level was set to check how many times this occurs without getting a very long log file
This method seems kind of pointless, since you can just as easily (or more easily) access these attributes on the object itself.
haha.  my java is shining through...  I debated this.  but your point is a good one.  fixed.
Similar to other comments below, you can use `.toggle()` to clean up this function
Afaik there isn't and maybe would be good in other tests / js files as well.
It could however break some things (e.g. 3th party scripts, relying on non-standard JS) so I didn't add it to other files.

Defining the `"use script"` within the scope of this function will also make sure that it doesn't influence other files.
well, the text part should contain something like

    * Your Gemfile is missing Rake (#1.1, #1.2)

the html part should contain, well, html :) so, something like:

    * <li>Your Gemfile is missing Rake (<a href="http://...">#1.1</a>)</li>

Wdyt?
`atomicStore(*cast(shared)&m_addr, cast(shared)m_addr.init);`
I thought we were going to use mocks for these tests
Typo meth~~**i**~~od
/**
	 * Constructs a DefaultLockRegistry with the supplied
	 * mask - the mask must have a value (2**n) - 1 where n
	 * is 1 to 31, creating a hash of 2**n locks.
	 * <p> Examples:
	 * <li>0x3ff (1023) - 1024 locks</li>
	 * <li>0xfff (4095) - 4096 locks</li>
	 * <p>
	 * @param mask 
	 */
Well, we always had method that did multiple operations. In fact almost all of them are and always have been. 
I guess it assumes that these operations are going to be executed as part of a transaction scope already defined by the user. So I am not sure how to answer that since adding it certainly changes the original design (which may not be such a bad idea after all)
You're still going to need to declare the passenger class somehow, though (since you remove the passenger class declaration)
No, it wasn't used at all (nothing inside the method definition changed). We pass the name to the connection object during initialization. `log_info` uses the value from its internal `@config` data.

As there is no need to pass the identifiers around, I removed them in order to simplify the code. Any objections?
This is good, thank you!
@christianv This is a more concise way of clearing floats.
please use camelCased names to follow the Sf2 coding standards
Why not pass the ShapePart directly, rather than its index?
I am running 0.8.* on linux right now just fine
-----Original Message-----
From: James Keshavarzi <notifications@github.com>
Date: Wed, 05 Dec 2012 12:23:42 
To: blackberry/BB10-Webworks-Packager<BB10-Webworks-Packager@noreply.github.com>
Reply-To: blackberry/BB10-Webworks-Packager <reply@reply.github.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

Seems weird to suggest 0.6.20 here when we are telling them to install 0.6.10 above. We suggest 0.6.10 above because that's the version that works on Windows. I suggest we leave it as 0.6.10 for now. When i do my 0.8.14 support pull request we can reference that version.

I will add the period.

---
Reply to this email directly or view it on GitHub:
https://github.com/blackberry/BB10-Webworks-Packager/pull/229/files#r2326306
This fixes the initial state being visible.
Hmm, that actually changed awhile back. Looks like the test.conf file has the correct settings, this must've just been wrong for awhile. I'll go ahead and update it to something that makes sense.
we will change jackalope doctrine dbal to 2.2.* as per your PR
just looking with @Seldaek why the tests are failing .. because your change looks ok to me
Of course there's a missing comma here, after `... = true`. Will update the PR.
Hi Uwe,

Visibilityreduced to package visibility.
XCODE_DEPS_TARGET_FOLDER member is now final.

Best regards,
M.

From: Uwe Krueger <notifications@github.com<mailto:notifications@github.com>>
Reply-To: sap-production/xcode-maven-plugin <reply@reply.github.com<mailto:reply@reply.github.com>>
Date: Wednesday, November 14, 2012 5:49 PM
To: sap-production/xcode-maven-plugin <xcode-maven-plugin@noreply.github.com<mailto:xcode-maven-plugin@noreply.github.com>>
Cc: SAP SAP <marcus.holl@sap.com<mailto:marcus.holl@sap.com>>
Subject: Re: [xcode-maven-plugin] BugFix: symbolicLinks inside project zip file. (#42)


In modules/xcode-maven-plugin/src/main/java/com/sap/prd/mobile/ios/mios/FolderLayout.java:

> @@ -33,29 +33,29 @@
>  {
>    public final static String LIBS_DIR_NAME = "libs";
>    public final static String HEADERS_DIR_NAME = "headers";
> -  private static String XCODE_DEPS_TARGET_FOLDER = "xcode-deps";
> +  static String XCODE_DEPS_TARGET_FOLDER = "xcode-deps";


Should at least be final. Why are the others public and this one package private?

—
Reply to this email directly or view it on GitHub<https://github.com/sap-production/xcode-maven-plugin/pull/42/files#r2125406>.
and btw, for ``true``, ``false`` and ``null``, there is shortcut methods for the default value.
Je vais retravailler dessus. Pour le moment, pour du prettyprint, il faut
utiliser `<pre class="prettyprint">` au lieu d'un bloc de code Markdown.
Mais ca oblige à échapper le code HTML à la main. Donc 2 possibilités:

- Un bout de Javascript qui détecte tous les blocs pre générés par Markdown
dans les posts, leur ajoute la classe prettyprint et appelle la fonction
prettyPrint()
- Un moyen d'ajouter une classe à un bloc de code Markdown, en Markdown.

A mon avis, la première solution est plus simple et plus réalisable. Elle
évite aussi d'avoir à se poser trop de questions: tous les blocs de code
sont automatiquement prettifiés. Est-ce que vous voyez un cas où ça
poserait problème? Au pire, on ne prettifie les pre que s'ils nont PAS la
classe "noprettyprint".

Le 18 septembre 2012 00:46, Cyril Lacôte <notifications@github.com> a écrit
:

> In _posts/2012-09-19-responsive-js-media-queries.md:
>
> > +	|_____|_  |___|__,|___|
> > +	        |_|
> > +
> > +Je suis d'accord, ce use-case n'est pas le plus commmun. Mais à quoi bon faire un blog post sur un sujet bateau, sinon?
> > +
> > +#CSS3 media queries
> > +
> > +CSS3, grâce aux *media queries*, permet de styler le contenu en fonction de caractéristiques d'affichage : ratio, device, ou dimensions. C'est ce que met en oeuvre [Twitter Bootstrap](http://twitter.github.com/bootstrap/scaffolding.html#responsive
> ) pour assurer le responsive. Ainsi, il adapte sa grille naturellement, transformant les colonnes en lignes quand la largeur ne suffit plus.
> > +
> > +Des classes CSS (`.visible-desktop`, `.hidden-phone`, etc...) vous permettent même de masquer ou afficher du contenu en fonction d'un type de device visé.
> > +
> > +Par exemple :
> > +
> > +	<span class="visible-desktop">Hello Desktop!</span>
> > +	<span class="visible-tablet">Hello Tablet!</span>
> > +	<span class="visible-phone">Hello Phone!</span>
>
> Je n'ai pas testé avec la PR #3<https://github.com/Ninja-Squad/ninja-squad.github.com/issues/3>de JB qui embarquait prettyprint. Je tenterai de modifier.
>
> —
> Reply to this email directly or view it on GitHub<https://github.com/Ninja-Squad/ninja-squad.github.com/pull/6/files#r1624090>.
>
>
Yeah, I will add a URI.decode_query so you can do:

    URI.decode_query URI.parse(uri).query

It will have the advantage of also handling encoded chars.
hello again cackeKey => cacheKey
author = 4 ?

Privilégier un where(:name => 'dbloader') de nouveau ;-)
Firstly, what is awful about it? It's a terrific structure for small and large apps. What would you recommend, instead?

Secondly, I've built the application structure in a way which more closely represents a large application for one reason: they'll have to work with a similar structure in Medley (vs the basic, easy-to-grok-for-a-newcomer structure you get in the Django getting started guide).
This looks like an (internal) property to me.
Ah, yes.  I was trimming out comments I had added about the DOMParser explosion and this comment got killed with it.  I'll put this back.
This check was also mis-indented -- I've fixed that too.
When setAllowedGrantType is called on contructor, $this->allowedGrantType is empty, since hydrate hasn't ... hydrate. So, setAllowedGrantType sets an unserialized_allowed_grant_type.

When hydrate comes, allowed_grant_type is set, but getAllowedGrantType never "deserialize" allowed_grant_type due to unserialized_allowed_grant_type existence.

merge is not possible (instead of +) because it doesn't override already existing value.
There do not appear to be any events in-between the meta tags, I think they can be removed
`Cancels`
- move out admin css
- remove old code that takes over everything and replace with theme function call
You should update your pull request (just checkin again to the same branch you submitted the pull from and github will propagate so we see updates in the PR).
End with dot, also on next line.
the child is in repo, so get the child's path
Thanks for the link. I hadn't seen that before, but it's a good one to know.