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.

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

[7.4.0] Verify in tests that coverage supports --nobuild_runfile_links #23345

Closed
wants to merge 1 commit into from

Conversation

bazel-io
Copy link
Member

Fixes #20577

Closes #22676.

PiperOrigin-RevId: 649065192
Change-Id: I0a57f580c1cb3a03184950e3e5ea24e9ef4962b4

Commit c800608

Fixes bazelbuild#20577

Closes bazelbuild#22676.

PiperOrigin-RevId: 649065192
Change-Id: I0a57f580c1cb3a03184950e3e5ea24e9ef4962b4
@bazel-io bazel-io requested a review from a team as a code owner August 19, 2024 21:05
@bazel-io bazel-io added team-Core Skyframe, bazel query, BEP, options parsing, bazelrc awaiting-review PR is awaiting review from an assigned reviewer labels Aug 19, 2024
@iancha1992 iancha1992 requested a review from c-mita August 19, 2024 21:10
@iancha1992 iancha1992 enabled auto-merge (squash) August 19, 2024 21:10
@iancha1992
Copy link
Member

@c-mita do you have fix for the presubmit errors?

@Ryang20718
Copy link

@iancha1992 @c-mita I don't think this cherry-pick is correct.

404d8d9 is the actual fix whereas the cherry-pick above is only for adding tests

@c-mita
Copy link
Member

c-mita commented Aug 28, 2024

Attempting to cherry-pick 404d8d9 into 7.4.0 results in quite a few merge conflicts that I don't feel comfortable resolving myself.

Assigning to @lberki to take a look if it's reasonable or not.

@Ryang20718
Copy link

@lberki small ping! would love to get this in 7.4 if possible

@meteorcloudy
Copy link
Member

@Ryang20718 If a commit causes too much merge conflicts, we may eventually decide not to cherry pick it. But feel free to give it a try if you desperately need it.

@Wyverald
Copy link
Member

Wyverald commented Oct 9, 2024

We don't have time for this in 7.4.0. If we ever do a 7.5.0, we can revisit. Closing.

@Wyverald Wyverald closed this Oct 9, 2024
auto-merge was automatically disabled October 9, 2024 21:07

Pull request was closed

@Wyverald Wyverald deleted the cp20577-7.4.0 branch October 9, 2024 21:08
@github-actions github-actions bot removed the awaiting-review PR is awaiting review from an assigned reviewer label Oct 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Core Skyframe, bazel query, BEP, options parsing, bazelrc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants