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

Updated hls.js documentation link in README.md #440

Merged
merged 2 commits into from
Jul 31, 2018
Merged

Updated hls.js documentation link in README.md #440

merged 2 commits into from
Jul 31, 2018

Conversation

borozcod
Copy link
Contributor

@borozcod borozcod commented Jul 9, 2018

Updated hls.js documentation link.

@borozcod
hls.js documentation update
5c3fa59 
Updated hls.js documentation link.
@cookpete
Copy link
Owner

Seems like the only change here is removing the #fine-tuning anchor to load the correct section of the page. Why remove it?

@borozcod
Copy link
Contributor Author

I believe they changed the /doc to /docs I can add the #fine-tuning at the end of the link and commit that change.

@cookpete
Copy link
Owner

Good spot @borozcod. Yeah if you add #fine-tuning back I'm happy to merge this up.

@borozcod
Updated Readme
57c0b40 
Added #fine-tuning to the end of the hls documentation link.
@borozcod
Copy link
Contributor Author

Committed that change. Thank you.

@cookpete cookpete merged commit 52d98ad into cookpete:master Jul 31, 2018
@cookpete
Copy link
Owner

Thanks!

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
@borozcod @cookpete