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

Add github action which pushes empty commit every month #128

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

c4rt0
Copy link
Member

@c4rt0 c4rt0 commented Dec 19, 2024

See: #118

@c4rt0 c4rt0 force-pushed the PR/add_maintenance branch from 4e30eb4 to ad2c68b Compare December 19, 2024 15:34
@c4rt0 c4rt0 requested a review from prestist December 19, 2024 15:35
runs-on: ubuntu-latest
steps:
- name: Checkout repository
uses: actions/checkout@v2
Copy link
Member

Choose a reason for hiding this comment

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

Looking at github actions it seems v2 is really outdated. Use actions/checkout@v4, see: https://github.com/actions/checkout/tree/v4

Copy link
Member Author

Choose a reason for hiding this comment

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

Thank you, I will address things this way from now on!

@travier
Copy link
Member

travier commented Feb 13, 2025

It might still be safer to make a PR instead of directly pushing to the repo. I've set the config so that the workflows can create PRs in the repository.

@c4rt0
Copy link
Member Author

c4rt0 commented Feb 13, 2025

It might still be safer to make a PR instead of directly pushing to the repo. I've set the config so that the workflows can create PRs in the repository.

Correct me if I'm wrong but if we are able to create PRs automatically, I'm pretty sure we can also close them the very same way.
I will look into that, thanks for the tip!

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.

3 participants