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

Fix markdown formatting / typo #152

Merged
merged 2 commits into from
Dec 14, 2018
Merged

Fix markdown formatting / typo #152

merged 2 commits into from
Dec 14, 2018

Conversation

agsimmons
Copy link
Contributor

No description provided.

@JustAMan JustAMan changed the base branch from master to develop December 14, 2018 16:10
Copy link
Contributor

@JustAMan JustAMan left a comment

Choose a reason for hiding this comment

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

Changes look good to me.
Please make a PR to develop branch next time as we've agreed that all commits should not go to master directly. Not sure if it was already properly fixed up in docs... @joshuaboniface, @anthonylavado, comments?

I've fixed this PR.

@agsimmons
Copy link
Contributor Author

Sorry about that, but CONTRIBUTING.md states to make text/documentation changes against master. Am I understanding that incorrectly?

@JustAMan
Copy link
Contributor

I think we changed the agreement yesterday but nobody updated the file.
Really sorry about that!

I'll let others chime in to say whether I'm right about that change anyway.

@anthonylavado
Copy link
Member

Oops! Yes, we didn’t update the docs yet to show that. Good catch! Will fix.

@anthonylavado anthonylavado merged commit 396f721 into jellyfin:develop Dec 14, 2018
nvllsvm pushed a commit that referenced this pull request Dec 27, 2018
Fix markdown formatting / typo
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.

3 participants