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

chore: archive crates-io-infra repository #1521

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

MarcoIeni
Copy link
Member

@MarcoIeni MarcoIeni commented Aug 14, 2024

I found this old private repo that seems empty. Should we delete this file and archive the repo?

@MarcoIeni MarcoIeni marked this pull request as ready for review August 14, 2024 08:04
@ehuss
Copy link
Contributor

ehuss commented Aug 14, 2024

Would it make sense to move it to the archive directory and have sync-team handle making it archived?

@Kobzol
Copy link
Contributor

Kobzol commented Aug 14, 2024

In general, we do mostly archive repositories, rather than delete them, yes. sync-team is not even able to delete repositories (this is by design, as it is an irreversible change).

@MarcoIeni MarcoIeni changed the title chore: remove crates-io-infra repository chore: archive crates-io-infra repository Aug 14, 2024
@MarcoIeni
Copy link
Member Author

MarcoIeni commented Aug 14, 2024

Archiving the repo was what I wanted to do, I didn't know we had an archive directory 😅
Thanks! 😁

I moved the file to the archive repo 👍

@MarcoIeni MarcoIeni merged commit 75490d3 into master Aug 19, 2024
2 checks passed
@rylev rylev deleted the remove-crates-io-infra-repo branch August 19, 2024 10:34
@jackh726
Copy link
Member

Per archiving policy, team access here should have been removed.

@MarcoIeni
Copy link
Member Author

Ooops,

fix: #1526

I added a validation rule: #1527 to be merged after the fix PR 👍

@Kobzol
Copy link
Contributor

Kobzol commented Aug 20, 2024

Remark: this repository is private, so after all access to it was removed, it is now essentially hidden from the world (maybe just the infra-admins see it). Not sure if that's a problem or not, just wanted to let it be known.

@MarcoIeni
Copy link
Member Author

As an infra-admin, I can see it 👍

Not sure if that's a problem or not, just wanted to let it be known.

I don't know either, if it's a problem for someone, we can discuss it 👍

Btw for context, this repo just had a readme with two lines.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants