Skip to content
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.

Sign up for GitHub

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

Jump to bottom

fix(IDX): check that bazel-out exists #4176

Merged
merged 2 commits into from
Feb 28, 2025
Merged

fix(IDX): check that bazel-out exists #4176

merged 2 commits into from
Feb 28, 2025

Conversation

nmattia
Copy link
Contributor

@nmattia nmattia commented Feb 28, 2025

This adds a check to make sure that bazel-out exists before we scan it for checksums. If bazel-out (a symlink) doesn't exist, then we create an empty SHA256SUMS.

@nmattia
fix(IDX): check that bazel-out exists
05af15b 
This adds a check to make sure that `bazel-out` exists before we scan it
for checksums. If `bazel-out` (a symlink) doesn't exist, then we create
an empty `SHA256SUMS`.
@github-actions github-actions bot added the fix label Feb 28, 2025
@nmattia nmattia marked this pull request as ready for review February 28, 2025 16:33
@nmattia nmattia requested a review from a team as a code owner February 28, 2025 16:33
@nmattia nmattia enabled auto-merge February 28, 2025 16:33
@github-actions github-actions bot added the @idx label Feb 28, 2025
@nmattia nmattia added this pull request to the merge queue Feb 28, 2025
Merged via the queue into master with commit 66592ee Feb 28, 2025
23 checks passed
@nmattia nmattia deleted the nm-bazel-out branch February 28, 2025 16:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@marko-k0 marko-k0 marko-k0 approved these changes

Assignees
No one assigned
Labels
fix @idx
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@nmattia @marko-k0