Skip to content

CI: add push actions; update action versions - #459

Open
rgwott wants to merge 1 commit into
ARM-software:mainfrom
rgwott:workflows_on_push
Open

CI: add push actions; update action versions#459
rgwott wants to merge 1 commit into
ARM-software:mainfrom
rgwott:workflows_on_push

Conversation

@rgwott

@rgwott rgwott commented Aug 27, 2026

Copy link
Copy Markdown
Contributor
  • Trigger tag/deploy workflows automatically on push/PR merge.
  • Update action versions to newer ones with Node 24.
  • Delete Latest release before creating a new Latest.

@CarolineConcatto CarolineConcatto left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why are ups updating the version?

Comment thread .github/workflows/ci.yml
jobs:
build-pdfs:
if: github.event_name == 'pull_request' || github.event_name == 'workflow_dispatch'
if: github.event_name == 'pull_request' || github.event_name == 'workflow_dispatch' || (github.event_name == 'push' && github.ref == 'refs/heads/main')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Why the pdf and the mark downs dont need to check :
github.repository == 'ARM-software/acle'

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Those can be executed in any case. The ones that deploy tags and the website have this safeguard so that forks don't execute them (even though they should not succeed).

@rgwott

rgwott commented Aug 27, 2026

Copy link
Copy Markdown
Contributor Author

@CarolineConcatto We are updating the versions because the image used until now are deprecated, using Node 20 (which has safety flaws). The recommended approach is to use the newer versions.

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