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

doc update #873

Merged
merged 1 commit into from
Dec 3, 2016
Merged

doc update #873

merged 1 commit into from
Dec 3, 2016

Conversation

thijstriemstra
Copy link
Contributor

No description provided.

@@ -102,7 +102,7 @@ grunt coverage
The HTML report can be found in `coverage/html/index.html`.

## Editing documentation
The homepage and the documentation are in the [`gh-pages` branch](https://github.com/katspaugh/wavesurfer.js/tree/gh-pages). Contributions to the documentation are especially welcome.
The homepage and documentation files are maintained in the [`gh-pages` branch](https://github.com/katspaugh/wavesurfer.js/tree/gh-pages). Contributions to the documentation are especially welcome.
Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Out of curiosity, what was unclear with the previous wording?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The original text was correct. The edited version is slightly clearer. I would only disagree with adding "are maintained." The existence of the files implies updates, edits and rewrites.

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, Mr America! 👍

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I guess I meant are maintained elsewhere aka a different branch. Should I add that @katspaugh?

Copy link
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think your current version is fine.

@katspaugh katspaugh merged commit bf370ed into master Dec 3, 2016
@katspaugh katspaugh deleted the thijstriemstra-patch-1 branch December 3, 2016 11:04
mspae pushed a commit to mspae/wavesurfer.js that referenced this pull request Aug 19, 2017
mspae pushed a commit that referenced this pull request Aug 20, 2017
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