-
Notifications
You must be signed in to change notification settings - Fork 137
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
Mark 101, 108, 113 as replaced, due to out of date #48
Comments
(reference http://ros.org/reps/rep-0122.html#id38 for load_manifest block) |
Ok, so for each of these:
In each case, replaced doesn't seem appropriate, maybe retired or archived or inactive. I don't know. Should they be explicitly replaced by nothing? That seems like a misuse of the mechanism. I would guess we should have an inactive state for retired REP's which became "active". Thoughts? |
If we don't want to introduce new states to the REP process we can just keep them as they are. May be adding a note add the beginning mentioning that they are referring to an EOL ROS distro would help? |
That's fine by me, any objections to just adding a message the beginning of each of these? |
Looking at the PEPs there's a section for Historical PEPs (in particular They added the historical sorted section here: On Tue, Jan 13, 2015 at 6:38 PM, William Woodall notifications@github.com
|
@tfoote I agree we should track the PEP system, but from the first link it seems unresolved still:
Historical seems to only apply with Information reps if it isn't active and it doesn't have Release Schedule in the title: https://hg.python.org/peps/file/59dfaaf19a14/pep0/output.py#l70 That seems pretty odd to me, and would mean that REP 101 would never be marked historical because of the title. It may also be possible to just modify our REP 0 to allow for explicitly setting something to historical. I'm not sure what we should do, lets continue discussion here and make a decision at the next ROS team meeting. |
We discussed this, we'll just add notes for now, and if someone has time we'll import upstream changes. |
Mark 101, 108, 113 as replaced, due to out of date
See:
The text was updated successfully, but these errors were encountered: