-
Notifications
You must be signed in to change notification settings - Fork 12
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 text cleanup #21
base: master
Are you sure you want to change the base?
Conversation
This strongly depends on font-face and font-size. I would really not optimize to this, as this will be a ping/pong between different people with different settings. Just my 2 cents here. |
We have no GitHub issue number 1210. ;-) These numbers come from openttdcoop, so they no longer make sense, unless we explicitly mention that this is an openttdcoop bug number (which might be overkill if you ask me).
Did you even read it? It literally says "END OF TERMS AND CONDITONS" above. This section is clearly addressed to the licensor, i.e. us. For the licensee, i.e. the user, this section is irrelevant. |
76 should be a safe line length for most font sizes tho. Many lines were overlong. This is about making the text readable without ugly horizontal scrollbar under DEFAULT settings and window size. |
Ah, d'oh. OK, no real objection then.
Yes, I did read it, but the point is still that I think I pretty much have always seen the GPL quoted in its entirety, including "how to apply to new programs", and I don't think we should be cutting that bit out. |
The license commit was dropped. |
This PR just clean up some stuff in the readme files:
I recommend to review per-commit.