Rename gh-actions-pin references to gh-actions-lock - #4
Merged
Conversation
There was a problem hiding this comment.
✅ Ready to approve
Changes are straightforward reference updates with only a minor nit in the updated test assertion’s string coverage.
Note: this review does not count toward required approvals for merging.
Pull request overview
Updates this repository’s documentation, schema metadata, generated bindings, and Go package comments/tests to reflect the CLI rename from gh-actions-pin to gh-actions-lockfile, keeping references consistent across the schema and consumer-facing docs.
Changes:
- Rename tool references in the JSON schema description and regenerate the embedded Go schema constant accordingly.
- Update README related-project link and development coordination note to point at
github/gh-actions-lockfile. - Update Go doc comment and the future-version error-message test to reference the new CLI name.
File summaries
| File | Description |
|---|---|
| schema/lockfile-v0.0.1.json | Updates the schema’s human-readable producer-tool description to the new CLI name. |
| README.md | Updates related project link and schema-change coordination note to the renamed CLI repo. |
| go/pkg/lockfile/schema_gen.go | Regenerates embedded schema string to match the updated schema JSON description. |
| go/pkg/lockfile/lockfile.go | Updates Parse godoc example consumer reference to the new CLI name. |
| go/pkg/lockfile/lockfile_test.go | Updates the “must not name a specific consumer tool” assertion to the new name. |
Copilot's findings
Files not reviewed (1)
- go/pkg/lockfile/schema_gen.go: Generated file
- Files reviewed: 4/5 changed files
- Comments generated: 1
Note
Your feedback helps us improve the quality of this feature.
Please use 👍 or 👎 to tell us whether this assessment is correct.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // must not name a specific wrapping CLI. Consumers append their own | ||
| // upgrade instructions off errors.Is(err, ErrFutureVersion). | ||
| assert.NotContains(t, msg, "gh-actions-pin", "library message must not name a specific consumer tool") | ||
| assert.NotContains(t, msg, "gh-actions-lockfile", "library message must not name a specific consumer tool") |
The CLI was renamed from gh-actions-pin to gh-actions-lock. Update schema description, README links, code comments, and the negative test assertion to match.
nodeselector
force-pushed
the
nodeselector/rename-actions-pin-to-lockfile
branch
from
June 14, 2026 19:28
8e210c2 to
53eb5c2
Compare
nodeselector
added a commit
that referenced
this pull request
Jul 8, 2026
…lockfile Rename gh-actions-pin references to gh-actions-lock
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The CLI was renamed from
gh-actions-pintogh-actions-lock. This updates all stale references in this repo to match:schema/lockfile-v0.0.1.json) -- the human-readable blurb that names the producing tool.schema_gen.go) -- regenerated from the updated schema JSON.lockfile.go-- example consumer name in theParsegodoc.lockfile_test.go-- the "library must not name a specific consumer tool" check now guards against the new name.