Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

ci: fix readme-stars #36

Merged
merged 5 commits into from
Nov 24, 2023
Merged

Conversation

tguichaoua
Copy link
Contributor

@tguichaoua tguichaoua commented Nov 23, 2023

  • update actions/checkout to v4
  • update stefanzweifel/git-auto-commit-action to v5
  • add write permission, required by stefanzweifel/git-auto-commit-action
  • fix run condition to run only if AOC_ENABLED has the value true
  • move run condition to job so it stop sooner
  • use variable for AOC_ENABLED since it's not a sensitive value and also because it's required to move the run condition on the job

@fspoettel fspoettel merged commit 751fde7 into fspoettel:main Nov 24, 2023
@fspoettel
Copy link
Owner

Thank you very much! 🎄

@tguichaoua tguichaoua deleted the ci/fix-readme-stars branch November 24, 2023 08:26
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants