Skip to content

ci: fix security-scans action reference broken by #394#400

Merged
mnencia merged 2 commits intomainfrom
dev/fix-security-scans-action
Feb 26, 2026
Merged

ci: fix security-scans action reference broken by #394#400
mnencia merged 2 commits intomainfrom
dev/fix-security-scans-action

Conversation

@NiccoloFei
Copy link
Contributor

@NiccoloFei NiccoloFei commented Feb 24, 2026

PR #394 accidentally reverted the security-scans action reference from the fully qualified syntax back to a local path, breaking the reusable workflow for external repositories like postgis-containers.

Restore the fully qualified reference and add protective comments on both action references to prevent future regressions.

@dosubot dosubot bot added size:XS This PR changes 0-9 lines, ignoring generated files. github_actions Pull requests that update GitHub Actions code labels Feb 24, 2026
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 24, 2026
@sxd sxd force-pushed the dev/fix-security-scans-action branch from 03f36b1 to a2d1af3 Compare February 26, 2026 16:36
@mnencia mnencia changed the title ci: reference security-scans action via fully qualified syntax ci: fix security-scans action reference broken by #394 Feb 26, 2026
NiccoloFei and others added 2 commits February 26, 2026 18:05
Signed-off-by: Niccolò Fei <niccolo.fei@enterprisedb.com>
Signed-off-by: Marco Nenciarini <marco.nenciarini@enterprisedb.com>
@mnencia mnencia force-pushed the dev/fix-security-scans-action branch from 689de2e to 7998dd8 Compare February 26, 2026 17:05
@mnencia mnencia merged commit 0e76f4f into main Feb 26, 2026
3 of 6 checks passed
@mnencia mnencia deleted the dev/fix-security-scans-action branch February 26, 2026 18:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code lgtm This PR has been approved by a maintainer size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants