-
Notifications
You must be signed in to change notification settings - Fork 14
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
Prominent notice that this repo is unmaintained #87
Conversation
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.
LGTM (apart from two tiny nitpicks). 👍
README.md
Outdated
@@ -32,6 +45,8 @@ To view a module documentation execute: | |||
|
|||
### manageiq_provider module | |||
|
|||
**Replacement**: upstream https://docs.ansible.com/ansible/devel/manageiq_provider_module.html |
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.
nitpick: should be consistent with the messages in all other files.
.github/CONTRIBUTING.md
Outdated
@@ -0,0 +1,9 @@ | |||
# OBSOLETE — Upstreaming into Ansible itself | |||
|
|||
This repository is no longer maintained. |
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.
nitpick: I'd put "no longer maintained" in bold, so it will be more prominent.
LGTM. |
@cben do you have permissions to close this repo for new PRs ? |
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.
LGTM,
I think you also need to close this for new PRs
Settings -> Danger Zone -> Archive this repo
Addressed comments. I'll let myself merge without further review because it's just text, and don't want to forget this for another 3 weeks :-/ I don't have permissions to archive this repo, but I also don't want to archive until we're done upstreaming. |
And by now ansible.com moved my cheese, all links 404... Fix soon. |
Both users and contributor might look at this repo and not realize Ansible itself is the new place. (Evidence: we'd got a couple contributions here - #79, #86).
Added prominent notice to README (and a CONTRIBUTING file too which should appear when trying to open issue/PR here).
I don't want to completely close this repo to PRs/issues for now, in case someone still needs a module from here and wants to fix a bug.
I do want to signal we'll spend minimal efforts here.
@elad661 @yaacov @snecklifter Please review
I've updated #85 with current upstreaming status & interface changes.
And I'm going through PRs & issues on this repo, hope to close most.