Skip to content

chore(ci): don't download chrome in bump-auxiliary-packages#2391

Merged
nirinchev merged 2 commits intomainfrom
ni/auxiliary-packages-changes
Mar 4, 2025
Merged

chore(ci): don't download chrome in bump-auxiliary-packages#2391
nirinchev merged 2 commits intomainfrom
ni/auxiliary-packages-changes

Conversation

@nirinchev
Copy link
Collaborator

This is a minor change that sets the PUPPETEER_SKIP_DOWNLOAD environment variable in bump-auxiliary-packages.yml as we don't really need chrome there. Additionally, it adds a description for the workflow and removes some redundant committing.

@nirinchev nirinchev requested a review from gagik March 3, 2025 21:39
Comment on lines -44 to -45
git add .
git commit --no-allow-empty -m "chore(release): bump packages for auxiliary release" || true
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

peter-evans/create-pull-request already commits all changes, so there's no reason for us to manually do it.

Copy link
Contributor

@gagik gagik left a comment

Choose a reason for hiding this comment

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

LGTM, can probably be extended to the publish workflow also

contents: none # We use the github app to checkout and create PR

env:
PUPPETEER_SKIP_DOWNLOAD: "true"
Copy link
Contributor

Choose a reason for hiding this comment

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

oh nice, same can be applied to publish-auxiliary-packages.yml

@nirinchev nirinchev added the no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion) label Mar 4, 2025
@nirinchev nirinchev merged commit fdd0fd1 into main Mar 4, 2025
61 of 65 checks passed
@nirinchev nirinchev deleted the ni/auxiliary-packages-changes branch March 4, 2025 11:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

no-title-validation Skips validation of PR titles (conventional commit adherence + JIRA ticket inclusion)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants