-
Notifications
You must be signed in to change notification settings - Fork 4
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
Document the status of the software (maintenance, ci builds, distro support..) #55
Comments
issues related to the #51 (comment) discussion |
Suggestion: add this as a section to existing readmes. One file per organisation (ie: |
yes, i was going to suggest this as well! Each README needs at least a travis build badge. We also need to turn on cron jobs in travis because i restarted a few repos that i still have access to and they recently failed due to dependency issues: https://travis-ci.org/ros-industrial-consortium/godel It would also be great to have a single page (possibly this one?) to get a quick overview as to what needs attention. It would be very difficult to go into each repo and view the status. I wouldn't mind taking on that task. |
IMHO all READMEs should state (for each branch!):
A consolidated overview of all repositories would be great. In |
@ipa-mdl I am agree with your comment, the main problem is that not all the repos are following this rule for the branches. At least I would like to have a consensus about the template, use it (that will imply update it) or not should be a decision of the repository maintainers. I updated my previous version for ros_canopen as template https://github.com/ipa-nhg/ros_canopen/blob/UnifyReadme/README.md , let me know your opinion and feel free to propose other format. In parallel I will create a common Readme file for all the ROS-I supported packages as a general overview, but probably simpler ( as @AustinDeric already said, this will be very difficult if we pretend to support all repos, all branches, all distros.... ) |
My 2 cents: the currently suggested content comes across as chaotic. That is not necessarily a problem with the content, but seems to be more a result of the limitations of the various markup formats used in the readmes. Markdown just doesn't really support showing this in a nice way, unless we want to go into embedding raw html. I'm still not entirely convinced that listing all of this information is actually beneficial (it is a duplicate of what is shown on the ROS wiki after all), but one of the few 'aesthetically pleasing' ways this sort of info is visualised is the moveit readme: no button in a row: no support. Button in a row? Supported, and build status is reflected by the button. Button is immediately a link to the job. Columns show supported ROS releases. Perhaps we can do something like that. I would also very much recommend the proposed content be shown after things like the repository overview and first few sections. This may be personal, but I'd rather first understand what a repository has to offer than be shown a (potentially very large) table with stuff I might not even be interested in at first. As to an overview of everything: something to figure out is where to host that. There are no organisation level readmes or descriptions (that allows sufficient space), so that would seem to require us to put it into a repository or potentially the ROS wiki. Alternatives could of course also be used. Edit: just saw ros-industrial/ros_canopen@1d76225: that seems already much better. |
For the individual READMES, lets keep open the discussion about the format. Probably @ipa-mdl will propose a nice alternative soon.... For everything, I can imagine the following options to host the overview:
About the content, we should investigate the way to have as less duplicated information as possible, I prefer always the option of having the information only once and link it. I hope I have time soon to come up with a proposal ... |
We can create a common file as overview of the status of all the ROS-I repos, similar to this template:
https://github.com/ipa320/cob_robots/blob/indigo_dev/README.md
Also can be part of each repo README.md
The text was updated successfully, but these errors were encountered: