-
Notifications
You must be signed in to change notification settings - Fork 314
chore: install everything #2596
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
base: canary
Are you sure you want to change the base?
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
🔒 Entelligence AI Vulnerability Scanner ✅ No security vulnerabilities found! Your code passed our comprehensive security analysis. |
Review Summary |
|
LGTM 👍 |
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.
💡 Codex Review
Here are some automated review suggestions for this pull request.
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting
| using: "composite" | ||
| steps: | ||
| - name: Setup mise | ||
| uses: jdx/mise-action@v2 | ||
| uses: jdx/mise-action@v3 | ||
| with: | ||
| install_args: "protoc-gen-go" | ||
| cache: true |
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.
Protoc-gen-go no longer installed in setup-tools action
Switching Setup Tools to jdx/mise-action@v3 removed the only install_args call, so the step now only sets up mise and cache without installing any tools. Nothing else in the workflows installs protoc-gen-go—setup-go still only installs Go itself—yet the Rust lint and build jobs rely on protoc-gen-go being available (see the comments around those steps and the build.rs lookup via mise which protoc-gen-go). The next runs of those jobs will fail when protoc-gen-go is missing from the PATH. Please keep an explicit install (either install: true or install_args: "protoc-gen-go") when upgrading to v3.
Useful? React with 👍 / 👎.
|
🌿 Preview your docs: https://boundary-preview-636f1432-fa53-41cf-8d02-bed2c953d4d8.docs.buildwithfern.com |
|
🌿 Preview your docs: https://boundary-preview-44219f37-23cb-4369-a4f1-8585b8290185.docs.buildwithfern.com |
|
🌿 Preview your docs: https://boundary-preview-ab167854-c046-43aa-84e8-94563214d400.docs.buildwithfern.com |
|
|
||
| - uses: gradle/actions/setup-gradle@v4 | ||
| - name: Setup build tooling | ||
| uses: gradle/actions/setup-gradle@v4 |
Check warning
Code scanning / CodeQL
Unpinned tag for a non-immutable Action in workflow Medium
Uses Step
|
🌿 Preview your docs: https://boundary-preview-38eb3af2-b173-4ae3-a3b1-cce410d5407d.docs.buildwithfern.com |
|
🌿 Preview your docs: https://boundary-preview-43907834-3df2-46f4-971c-804ef35e6890.docs.buildwithfern.com |
|
🌿 Preview your docs: https://boundary-preview-a3198294-ae06-4c98-ace7-e61b4e276743.docs.buildwithfern.com |
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
|
🌿 Preview your docs: https://boundary-preview-80f6cbfe-377a-491b-9482-be96f80cb1c9.docs.buildwithfern.com |
|
🌿 Preview your docs: https://boundary-preview-dc88ee8a-2ea6-4ccd-89e6-10cc21f32fea.docs.buildwithfern.com |
|
🌿 Preview your docs: https://boundary-preview-fbab05be-182d-416b-8c35-b0e0ce766130.docs.buildwithfern.com |
No description provided.