Skip to content

ci: don't assume full shell for postUpgradeTasks#3352

Merged
alan-agius4 merged 1 commit intoangular:mainfrom
hybrist:dont-assume-shell-for-post-upgrade-tasks
Jan 6, 2026
Merged

ci: don't assume full shell for postUpgradeTasks#3352
alan-agius4 merged 1 commit intoangular:mainfrom
hybrist:dont-assume-shell-for-post-upgrade-tasks

Conversation

@hybrist
Copy link
Copy Markdown
Contributor

@hybrist hybrist commented Jan 6, 2026

Newer versions of renovate are more conservative about allowing arbitrary shell primitives inside of commands to prevent accidental escapes from the allowlist. This changes the one place where we do want a "fancy" shell feature to explicitly opt into execution under bash. The command in question should be static enough to be safe.

Newer versions of renovate are more conservative about allowing
arbitrary shell primitives inside of commands to prevent accidental
escapes from the allowlist. This changes the one place where we do
want a "fancy" shell feature to explicitly opt into execution under
bash. The command in question should be static enough to be safe.
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label Jan 6, 2026
@hybrist hybrist added the action: merge The PR is ready for merge by the caretaker label Jan 6, 2026
@alan-agius4 alan-agius4 merged commit 87ed54d into angular:main Jan 6, 2026
8 checks passed
@alan-agius4
Copy link
Copy Markdown
Contributor

This PR was merged into the repository. The changes were merged into the following branches:

@angular-automatic-lock-bot
Copy link
Copy Markdown

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot Bot locked and limited conversation to collaborators Feb 6, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants