Skip to content

ci: update the Update Dependencies workflow #5175

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

Merged
jamescrosswell merged 4 commits into from
May 18, 2026
+6 −1
Merged

ci: update the Update Dependencies workflow #5175

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
bee386a
ci: update the Update Dependencies workflow
Flash0ver Apr 30, 2026
9f593e9
Merge branch 'main' into chore/update-workflow-update-deps
Flash0ver May 18, 2026
e0b2cfe
ci: don't update dependencies after each push to main
Flash0ver May 18, 2026
9587515
ci: remove note about CHANGELOG conflicts
Flash0ver May 18, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions .github/workflows/update-deps.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,18 @@ on:
# Run every day.
schedule:
- cron: "0 3 * * *"
# And on every PR merge so we get the updated dependencies ASAP, and to make sure the changelog doesn't conflict.
push:
branches:
- main

@Flash0ver Flash0ver Apr 30, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

question: add or revert?

It's part of the template/example: https://github.com/getsentry/github-workflows/tree/main/updater

On the one hand, we do daily updates already,
on the other hand, it wouldn't really hurt either.

@jamescrosswell jamescrosswell May 13, 2026

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Every push seems like an odd/arbitrary trigger. I think daily is fine.

@Flash0ver Flash0ver May 18, 2026

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

reverted via e0b2cfe

# Allow a manual trigger to be able to run the update when there are new dependencies or after a PR merge to resolve CHANGELOG conflicts.
workflow_dispatch:

permissions:
contents: write # To modify files and create commits
pull-requests: write # To create and update pull requests
actions: write # To cancel previous workflow runs

jobs:
deps:
name: ${{ matrix.name }}
Expand Down
Loading