Skip to content

ci: Weekly lockfile generation CI job#2966

Merged
sl0thentr0py merged 1 commit into
masterfrom
neel/dep-pinning
Jun 10, 2026
Merged

ci: Weekly lockfile generation CI job#2966
sl0thentr0py merged 1 commit into
masterfrom
neel/dep-pinning

Conversation

@sl0thentr0py

@sl0thentr0py sl0thentr0py commented Jun 10, 2026

Copy link
Copy Markdown
Member

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 2 potential issues.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 3442de1. Configure here.

Comment thread .github/workflows/update_lockfiles.yml
Comment thread .github/workflows/update_lockfiles.yml
Comment thread .github/workflows/update_lockfiles.yml Outdated
Comment thread .github/workflows/update_lockfiles.yml Outdated
@linear-code

linear-code Bot commented Jun 10, 2026

Copy link
Copy Markdown

RUBY-180

Comment thread .github/workflows/update_lockfiles.yml

@dingsdax dingsdax 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.

lgtm

I wonder though if this could by DRY-ed; steps are always the same (checkout, setup, bundlce, lock, upload) only gem name, matrix and lockfile name change, extracting the gen-* steps into a reusable workflow?

or separate workflow files per gem?

@sl0thentr0py

Copy link
Copy Markdown
Member Author

will do later

@sl0thentr0py sl0thentr0py merged commit 783e823 into master Jun 10, 2026
285 of 286 checks passed
@sl0thentr0py sl0thentr0py deleted the neel/dep-pinning branch June 10, 2026 15:26
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