Skip to content

chore(ci): remove frozen-lockfile and CI=true from workflows#1139

Merged
John-David Dalton (jdalton) merged 1 commit intomainfrom
chore/remove-frozen-lockfile-ci
Mar 31, 2026
Merged

chore(ci): remove frozen-lockfile and CI=true from workflows#1139
John-David Dalton (jdalton) merged 1 commit intomainfrom
chore/remove-frozen-lockfile-ci

Conversation

@jdalton
Copy link
Copy Markdown
Contributor

@jdalton John-David Dalton (jdalton) commented Mar 31, 2026

Remove redundant frozen-lockfile: 'true' inputs (the older install action accepted this but it's no longer needed) and drop the explicit CI: 'true' env var from the weekly-update apply-updates step (it caused pnpm to use frozen-lockfile during dep updates, breaking the workflow).


Note

Low Risk
Low risk workflow-only change that relaxes install/update strictness; potential impact is limited to CI determinism and dependency update behavior.

Overview
Removes redundant frozen-lockfile inputs from the SocketDev/socket-registry install action across ci.yml, provenance.yml, and weekly-update.yml.

Adjusts the weekly dependency update job by dropping CI: 'true' from the Claude update step so dependency update runs don’t implicitly force frozen-lockfile behavior.

Written by Cursor Bugbot for commit 2385647. Configure here.

Copy link
Copy Markdown

@cursor cursor bot left a comment

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 1 potential issue.

Fix All in Cursor

Bugbot Autofix is ON. A cloud agent has been kicked off to fix the reported issue.

Comment @cursor review or bugbot run to trigger another review on this PR

@jdalton John-David Dalton (jdalton) merged commit b5479b0 into main Mar 31, 2026
7 of 9 checks passed
@jdalton John-David Dalton (jdalton) deleted the chore/remove-frozen-lockfile-ci branch March 31, 2026 21:12
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