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

Fixes path to the test file #366

Merged
merged 2 commits into from
Jan 9, 2017
Merged

Fixes path to the test file #366

merged 2 commits into from
Jan 9, 2017

Conversation

cstuder
Copy link
Contributor

@cstuder cstuder commented Oct 10, 2016

Minor documentation patch.

@maggiepint
Copy link
Member

Can you add src to the other path as well? that's what people should be modifying in both cases.

@cstuder
Copy link
Contributor Author

cstuder commented Oct 18, 2016

Added the correct second path. I've had overlooked that, thanks for the heads up.

@maggiepint
Copy link
Member

Any chance on a CLA signature? Would like to merge this.

@cstuder
Copy link
Contributor Author

cstuder commented Jan 9, 2017

Sorry, didn't see the missing check. I just signed...

@maggiepint maggiepint merged commit c3669b1 into moment:master Jan 9, 2017
@cstuder cstuder deleted the patch-1 branch January 9, 2017 08:16
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@cstuder @maggiepint