-
Notifications
You must be signed in to change notification settings - Fork 157
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.
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
feat: add missing vendored tool version commands #2232
Conversation
✅ Deploy Preview for zarf-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like you'll need to run make docs-and-schema
to update the CLI docs but otherwise the code looks good!
Will need |
ce85228
to
163477a
Compare
163477a
to
ad8b98f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few more changes - also while the commit is unsigned - its unverified which may cause issues (partially verified commits can be merged but generally our repo settings will reject unverified ones)
0ad110f
to
5dd2dfd
Compare
6fd8bd6
to
8a4aca8
Compare
@Racer159 resigned the commit with a GPG key. Good for another check and 🚢 |
Signed-off-by: Eddie Zaneski <eddiezane@gmail.com>
71fc5a4
to
1ea4782
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Description
This PR adds missing version commands to the bundled tools.
Related Issue
Fixes #2142
Type of change
Checklist before merging