Skip to content

Bazel more blurbs#7612

Merged
maliberty merged 4 commits into
The-OpenROAD-Project:masterfrom
Pinata-Consulting:bazel-more-blurbs
Jun 18, 2025
Merged

Bazel more blurbs#7612
maliberty merged 4 commits into
The-OpenROAD-Project:masterfrom
Pinata-Consulting:bazel-more-blurbs

Conversation

@oharboe
Copy link
Copy Markdown
Collaborator

@oharboe oharboe commented Jun 18, 2025

No description provided.

oharboe added 2 commits June 18, 2025 17:26
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@oharboe oharboe force-pushed the bazel-more-blurbs branch from d635ca7 to 87af151 Compare June 18, 2025 15:28
@oharboe oharboe requested a review from maliberty June 18, 2025 15:28
@github-actions
Copy link
Copy Markdown
Contributor

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

1 similar comment
@github-actions
Copy link
Copy Markdown
Contributor

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

Comment thread docs/user/Bazel.md Outdated
Comment thread docs/user/Bazel.md Outdated
Signed-off-by: Øyvind Harboe <oyvind.harboe@zylin.com>
@github-actions
Copy link
Copy Markdown
Contributor

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

@maliberty maliberty enabled auto-merge June 18, 2025 16:34
@oharboe
Copy link
Copy Markdown
Collaborator Author

oharboe commented Jun 18, 2025

@maliberty merge manually? I just fixed a trivial merge conflict since you approved.

@github-actions
Copy link
Copy Markdown
Contributor

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

@maliberty
Copy link
Copy Markdown
Member

Its still set to automerge

@oharboe
Copy link
Copy Markdown
Collaborator Author

oharboe commented Jun 18, 2025

Its still set to automerge

And then it will go in even if I created more commits after you approved?

@maliberty maliberty merged commit e743761 into The-OpenROAD-Project:master Jun 18, 2025
11 checks passed
@oharboe oharboe deleted the bazel-more-blurbs branch June 28, 2025 04:39
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.

2 participants