Skip to content
This repository was archived by the owner on Nov 30, 2018. It is now read-only.

Conversation

aesnyder
Copy link

No description provided.

Aaron Ellis Snyder added 2 commits November 6, 2014 11:22
it really is default, just no specs are run.
risky, but no wifi on flight
@aesnyder
Copy link
Author

sorry the last commit does more than just namespace, but breaking it up is something I just don't have time for right now :/

@aesnyder aesnyder added PR'd and removed in progress labels Nov 11, 2014
@cthrax
Copy link
Contributor

cthrax commented Nov 11, 2014

Awesome commit, thanks for the work!

@nmccready
Copy link
Contributor

👍

1 similar comment
@gitnik
Copy link
Contributor

gitnik commented Nov 12, 2014

👍

@aesnyder
Copy link
Author

My pleasure!

aesnyder pushed a commit that referenced this pull request Nov 12, 2014
@aesnyder aesnyder merged commit 1eead63 into master Nov 12, 2014
@aesnyder aesnyder removed the PR'd label Nov 12, 2014
@cthrax
Copy link
Contributor

cthrax commented Nov 12, 2014

It occurred to me that one of the goals of this change was to stop modifying the prototype of String, but I think we still have that code in there, unless I just missed it in the diff.

@aesnyder
Copy link
Author

Yeah I've just been hitting it in waves - eventually we will eradicate it all

@cthrax
Copy link
Contributor

cthrax commented Nov 12, 2014

Gotcha, you're on it. Just thought I would mention it. Thanks again!

@aesnyder
Copy link
Author

:) no problem

@nmccready nmccready deleted the aes/remove-namespace-squashed branch January 29, 2015 16:16
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants