-
Notifications
You must be signed in to change notification settings - Fork 521
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
V2 : general cleanup of readme files #549
Conversation
TO DO:
|
During the updates, I'd like to make some adjustments to the README files. For the UI modules, these will have full information (README, LICENSE, CONTRIBUTING)
For component modules, these will have limited information (README)
|
@kaosat-dev what do you think about the above? one question... should modeling have LICENSE? CONTRIBUTING? |
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.
@z3dev : a solid set of changes, thanks !
I do have a few notes, and mostly a thought on how to save us maintenance work : perhaps the 'contributing' document should be moved to the root of the repo, and slightly changed so that it is valid for ALL UI packages (without the need of have 99% of the same text for all of them, in each folder), any specifics can be put into the README so there is only one main manual document to maintain for each .
What do you think ?
Sounds good to me. Less documents to maintain is better. I’ll merge the contributing documents that we have now, and update the links in the package specific readme documents. I think just one LICENSE is also best. Again, one document is easily maintained. And all the readme documents just link to the main one. |
corrected all package readme files to link to the LICENSE
…r to CONTRIBUTING document
So... just to get this correct... as I think some of the documentation needs some changes.
|
@z3dev yes, that is exactly it ! |
@kaosat-dev please review again. it's probably best to look at the results, not the differences. this is taking too much time, and will probably never stop. |
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.
Sorry @z3dev , while it is good overall, there are some things still worth tweak, almost there !
round one of small changes completed. :) @kaosat-dev back to you! |
These are general changes to the README documents for each package.
All Submissions: