-
Notifications
You must be signed in to change notification settings - Fork 11
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
Archive unused repos #67
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.
approve in concept but something about the config needs to change, I think maybe it doesn't like empty users? @galargh probably either something for you to fix in the permissions checking code or something obvious to fix in the config here?
Thanks @rvagg ! I should have checked to make sure the checks pass first. We'll figure out how to get that passing and then merge. |
The following access changes will be introduced as a result of applying the plan: Access Changes
|
Before merge, verify that all the following plans are correct. They will be applied as-is after the merge. Terraform plansipld
|
Noticed some repos that should probably be archived.
Summary
The are some repos that no longer look to be used. They were identified while going through #65. I acted in #66, but want to apply this subset of changes to master and also remove permissions of the archived repos.
Why do you need this?
Not needed, but it's for declutter.
It's also not a one-way door. Repos can be unarchived if needed.
What else do we need to know?
None
DRI: myself
Reviewer's Checklist