Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Copy edit comment text of application.js #12522

Closed
wants to merge 1 commit into from

Conversation

sjsyrek
Copy link

@sjsyrek sjsyrek commented Oct 27, 2015

I made corrections to punctuation, fixed several spelling mistakes,
added words that were accidentally omitted, and replaced two uses of
the obscure word “orthogonal” with the more common “independent.” Also
made some corrections to grammar and spacing and revisions for the sake
of consistent style (to the extent I could determine it).

I made corrections to punctuation, fixed several spelling mistakes,
added words that were accidentally omitted, and replaced two uses of
the obscure word “orthogonal” with the more common “independent.” Also
made some corrections to grammar and spacing and revisions for the sake
of consistent style (to the extent I could determine it).
@homu
Copy link
Contributor

homu commented Dec 8, 2015

☔ The latest upstream changes (presumably #12685) made this pull request unmergeable. Please resolve the merge conflicts.

@rwjblue
Copy link
Member

rwjblue commented Apr 10, 2016

Thank you for your time on this! I'm going to go ahead and close this for now as it seems to have gone stale. I'm happy to reopen if you would like to pick this back up again.

@rwjblue rwjblue closed this Apr 10, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants