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

Auto archive sections #583

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

AngelOnFira
Copy link
Member

This PR adds a GitHub Action that will check repos of projects once a week to see if new ones should be archived. It can be run manually, and has a check to not run if there are already-open PRs, so that it won't clutter the PR list over time.

You can see an example of the PR here: AngelOnFira#6

This should help with #95.

Requires that #582 is merged.

@AngelOnFira
Copy link
Member Author

I accidentally ran CI on this as the PR, but it threw an error. It does work when it's on my repo, so I think it should work fine once it's merged. I didn't change any settings on my end to have it work, so we'll see.

image

Copy link
Collaborator

@17cupsofcoffee 17cupsofcoffee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This seems like a smart change - I think automating stuff like this is the only way we'll ever be able to keep the site from getting stale 🙂

There might be additional criteria we can add for archiving crates (and I think in the long term, we probs should start deleting things that have been archived for a while), but we'll see how things look after this first pass.

My only comments would be:

  • I think we need to rebase now that Zola upgrade is merged.
  • It might be a good idea to have the job run on a different day to the weekly rebuild of the site, to reduce risk of us getting rate-limited.

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.

2 participants