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

README.md: fix links, line wraps #81

Closed
wants to merge 1 commit into from
Closed

README.md: fix links, line wraps #81

wants to merge 1 commit into from

Conversation

keikoro
Copy link

@keikoro keikoro commented Oct 2, 2017

(removed manual line wrapping because keeping it in order when editing files collaboratively is difficult)

@XaF
Copy link
Owner

XaF commented Nov 17, 2017

I'm not sure I agree with the removal of the line wrapping, as I feel like a README file should be readable easily even without it being rendered in Markdown, and even without an editor that does wrapping for you. Reading the file as-is in a really large console wouldn't be comfortable!
Can you please refactor the other changes without the removal of the wrapping ?

@keikoro
Copy link
Author

keikoro commented Nov 17, 2017

I've never seen manual line breaks in READMEs before – except maybe in a handful of Python projects where the README got derived from another file format (rst) – so would assume most people use READMEs differently (or know how to set their console's width), but sure, can refactor these changes.

@keikoro
Copy link
Author

keikoro commented Nov 18, 2017

See update from my side re: word wrap/line breaks in #82.

@XaF
Copy link
Owner

XaF commented Apr 16, 2018

TraktForVLC 2.x makes that PR obsolete - #88

@XaF XaF closed this Apr 16, 2018
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