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.

Sign up for GitHub

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

Jump to bottom

Fix link to CCRMA #60

Merged
merged 1 commit into from
Jan 15, 2018
Merged

Fix link to CCRMA #60

merged 1 commit into from
Jan 15, 2018

Conversation

endolith
Copy link
Contributor

@endolith endolith commented Jan 15, 2018 •
edited by bmcfee
Loading

This change is Reviewable

@bmcfee
Copy link
Owner

bmcfee commented Jan 15, 2018

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

@bmcfee
Copy link
Owner

bmcfee commented Jan 15, 2018

LGTM, thanks for catching that!

@bmcfee
Copy link
Owner

bmcfee commented Jan 15, 2018

Before I merge this, could you also fix the link in docs/index.rst?

@endolith
Copy link
Contributor Author

Isn't that what I did?

@bmcfee
Copy link
Owner

bmcfee commented Jan 15, 2018

Sorry, my bad! I misread this as fixing the README (which already was correct).

We're okay to merge then, right?

@endolith
Copy link
Contributor Author

Yes :)

@bmcfee bmcfee merged commit 3e76335 into bmcfee:master Jan 15, 2018
@bmcfee
Copy link
Owner

bmcfee commented Jan 15, 2018

Done. Thanks!

@endolith endolith deleted the patch-1 branch January 15, 2018 20:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@endolith @bmcfee