-
Notifications
You must be signed in to change notification settings - Fork 238
Move documentation files to docs folder #2283
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
Conversation
|
There are certain things that MUST be in the top level: README.md Anything more than those two could live elsewhere, so long as README.md lists them and says where they are and what they're for. |
|
Yes. I didn't move the README file. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- Should this get a changelog entry? It's a user-/contributor-visible change, so I'd say yes?
-
tools/create-translation-issues.shneeds updating wrtTRANSLATING.md - jamulussoftware/jamuluswebsite:
README.mdneeds updating wrtTRANSLATING.md(separate PR as it's another repo)
|
Not sure if it's worth a change log entry. But we can, of course |
|
CHANGELOG: Move documentation files into docs/ folder to clean up root folder. |
hoffie
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good to have this. Thanks! :)
|
Seems as if the author information is lost due to the move. |
|
What do you mean? |
|
I mean in the GitHub gui. It lists me as only contributor for the three files |
|
Hm, too bad, but I don't think there's anything we could have done differently. Looks like Github lacks support for this, so I guess we'll have to live with it. |
|
ok. so we'll need to live with it. |
Short description of changes
Moves/deletes some files to a newly created folder. Maybe we need to overthink the documentation structure. Having too many files in the root folder is confusing.
Context: Fixes an issue?
Not yet.
Does this change need documentation? What needs to be documented and how?
Status of this Pull Request
Proof of concept (not to be merged soon)
What is missing until this pull request can be merged?
Discussion
Checklist
Unrelated