Skip to content

ci: fix readme-stars #36

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Nov 24, 2023
Merged

ci: fix readme-stars #36

merged 5 commits into from
Nov 24, 2023

Conversation

tguichaoua
Copy link
Contributor

@tguichaoua tguichaoua commented Nov 23, 2023 •
edited
Loading

  • 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@fspoettel fspoettel fspoettel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@tguichaoua @fspoettel