Skip to content

Remove MODULE.bazel.lock for downstream testing.#10042

Merged
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
hzeller:feature-20260402-no-demo-lock
Apr 7, 2026
Merged

Remove MODULE.bazel.lock for downstream testing.#10042
maliberty merged 1 commit intoThe-OpenROAD-Project:masterfrom
hzeller:feature-20260402-no-demo-lock

Conversation

@hzeller
Copy link
Copy Markdown
Collaborator

@hzeller hzeller commented Apr 2, 2026

This file is not relevant for testing if a downstream compile would work, but a maintenance burden.

This file is not relevant for testing if a downstream
compile would work, but a maintenance burden.

Signed-off-by: Henner Zeller <h.zeller@acm.org>
@gemini-code-assist
Copy link
Copy Markdown
Contributor

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Apr 2, 2026

clang-tidy review says "All clean, LGTM! 👍"

@hzeller
Copy link
Copy Markdown
Collaborator Author

hzeller commented Apr 7, 2026

@oharboe Looks good ?

@oharboe
Copy link
Copy Markdown
Collaborator

oharboe commented Apr 7, 2026

🤷 I don't recall anything about this...

@hzeller
Copy link
Copy Markdown
Collaborator Author

hzeller commented Apr 7, 2026

@oharboe
Copy link
Copy Markdown
Collaborator

oharboe commented Apr 7, 2026

I asked: it seemed useful at the time.

@maliberty maliberty merged commit fa69edb into The-OpenROAD-Project:master Apr 7, 2026
15 checks passed
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.

3 participants