Skip to content
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

setup: archive old setup scripts #1505

Closed
wants to merge 1 commit into from

Conversation

refack
Copy link
Contributor

@refack refack commented Sep 26, 2018

Move /setup/ to /backup/archived-machine-setup/ doc\reference-setup-scripts

Fixes: #1277

@refack refack self-assigned this Sep 26, 2018
@richardlau
Copy link
Member

Why not branch/tag the repository and then delete the scripts instead of moving them if they are no longer used?

@refack
Copy link
Contributor Author

refack commented Sep 26, 2018

Why not branch/tag the repository and then delete the scripts instead of moving them if they are no longer used?

They represent the status quo, and we might want to use them as reference. They are just probably inoperable.

mmarchini

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

joaocgreis

This comment was marked as off-topic.

@refack
Copy link
Contributor Author

refack commented Sep 27, 2018

Even if unusable, I try to keep www updated when changes are made, and I see at least @rvagg does as well. It's not great but it's the best place we have to document changes, so I'm reluctant to change it to a folder named archive that doesn't encourage changes.

I get what you are saying, and I agree. That was also my understanding of the situation, and indeed it should be reflected in folder names. I'll try to come up with alternative suggestions.

@joaocgreis
Copy link
Member

Perhaps just move everything but www, and add a setup/README that makes this clear. Or rename setup to something like drafts.

@refack
Copy link
Contributor Author

refack commented Nov 17, 2018

@joaocgreis now moved to doc\reference-setup-scripts

@joaocgreis
Copy link
Member

Having it in doc seems strange to me. I don't see a need to remove setup/, but I won't block this if you do.

It would be good to have explicit approval from @rvagg since he also uses this folder.

@rvagg
Copy link
Member

rvagg commented Dec 1, 2018

I might try and move www into somewhere in ansible because it is still relevant, I can't speak for aix and some other bits in there but we can probably just remove it all, git is our "archive"

@sam-github
Copy link
Contributor

It looks from conversation that there was some objections, and now it has no champion. Closing, but it can be reopened if someone wants to take it up again.

@sam-github sam-github closed this Jan 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

State of Ansible
7 participants