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

Script automation #26

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

Script automation #26

wants to merge 2 commits into from

Conversation

bircni
Copy link

@bircni bircni commented Sep 28, 2024

On every commit into main this script now gets triggered to check if there are new icons available

@amPerl
Copy link
Owner

amPerl commented Oct 9, 2024

good idea as a concept, but I'm not sure if per-commit is very useful

consider the usual scenario of this crate mainly updating to the next version of egui:
the commits could be as far as months in between and that means it won't run at all for months, and then it'll run for a bunch of commits in a burst, and then potentially not again for months.

maybe once a week or every night would work better? i'm not sure how much that adds up to in actions-hours. is it possible to have an external trigger of some sort when another repo (such as phosphor core) is updated?

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