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 #372

Merged
merged 2 commits into from Jun 5, 2016
Merged

Copy-edit #372

merged 2 commits into from Jun 5, 2016

Conversation

slashme
Copy link
Contributor

@slashme slashme commented Apr 22, 2016

Minor grammar and style edits.

Minor grammar and style edits.
@stantond
Copy link

stantond commented Jun 5, 2016

Small misspelling on line 85, beofre

@slashme
Copy link
Contributor Author

slashme commented Jun 5, 2016

D'oh! Muphry's law in action :-]

@slashme
Copy link
Contributor Author

slashme commented Jun 5, 2016

Let me know whether you want me to redo the pull request on the current version, so that this will pass the tests, or whether you'll just apply the patch and ignore the failed check.

@slashme slashme closed this Jun 5, 2016
@slashme slashme deleted the patch-1 branch June 5, 2016 13:58
@slashme slashme restored the patch-1 branch June 5, 2016 14:01
@slashme
Copy link
Contributor Author

slashme commented Jun 5, 2016

OK, I've just checked, and nothing has actually changed on the master branch since I did these copy-edits, so the test that fails has nothing to do with this patch, and there is no new code to merge before applying this patch: reopening pull request.

@slashme slashme reopened this Jun 5, 2016
@poswald
Copy link
Collaborator

poswald commented Jun 5, 2016

I had also done some documentation edits on this branch but if they collide I'm more than willing to update them there. I would love anyone to look that PR over.

@poswald poswald merged commit cf6d333 into jazzband:master Jun 5, 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.

None yet

3 participants