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

Clean up cc_shared_library runfiles test #21880

Closed
wants to merge 3 commits into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Apr 3, 2024

Simplifies the test of cc_shared_library's runfiles behavior using new rules_testing features, which also result in better failure messages.

Also format BUILD.builtin_test with buildifier to simplify future contributions.

Work towards #21833

@fmeum fmeum requested a review from oquenchil April 3, 2024 14:29
@fmeum fmeum added team-Rules-CPP Issues for C++ rules awaiting-review PR is awaiting review from an assigned reviewer labels Apr 3, 2024
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Apr 3, 2024
@oquenchil oquenchil requested review from pzembrod and removed request for oquenchil April 11, 2024 12:25
@fmeum
Copy link
Collaborator Author

fmeum commented Apr 12, 2024

@bazel-io fork 7.2.0

@fmeum
Copy link
Collaborator Author

fmeum commented Apr 12, 2024

@pzembrod Do you want me to resolve the conflict in MODULE.bazel.lock or are you doing that during import? Also, does this need the awaiting-merge label?

@pzembrod pzembrod added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Apr 12, 2024
@pzembrod
Copy link
Contributor

@fmeum Thanks for the awaiting-merge reminder; done.
And the conflict in MODULE.bazel.lock will indeed be resolved during import.

@fmeum fmeum deleted the 21833-runfiles-test branch April 19, 2024 11:16
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request Apr 19, 2024
Simplifies the test of `cc_shared_library`'s runfiles behavior using new rules_testing features, which also result in better failure messages.

Also format `BUILD.builtin_test` with buildifier to simplify future contributions.

Work towards bazelbuild#21833

Closes bazelbuild#21880.

PiperOrigin-RevId: 626315579
Change-Id: Ib7b3d1367586ba7215132d6a9f3711f4ae0a9b09
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 26, 2024
iancha1992 pushed a commit to iancha1992/bazel that referenced this pull request Apr 26, 2024
Simplifies the test of `cc_shared_library`'s runfiles behavior using new rules_testing features, which also result in better failure messages.

Also format `BUILD.builtin_test` with buildifier to simplify future contributions.

Work towards bazelbuild#21833

Closes bazelbuild#21880.

PiperOrigin-RevId: 626315579
Change-Id: Ib7b3d1367586ba7215132d6a9f3711f4ae0a9b09
fmeum added a commit to fmeum/bazel that referenced this pull request May 10, 2024
Simplifies the test of `cc_shared_library`'s runfiles behavior using new rules_testing features, which also result in better failure messages.

Also format `BUILD.builtin_test` with buildifier to simplify future contributions.

Work towards bazelbuild#21833

Closes bazelbuild#21880.

PiperOrigin-RevId: 626315579
Change-Id: Ib7b3d1367586ba7215132d6a9f3711f4ae0a9b09
Kila2 pushed a commit to Kila2/bazel that referenced this pull request May 13, 2024
Simplifies the test of `cc_shared_library`'s runfiles behavior using new rules_testing features, which also result in better failure messages.

Also format `BUILD.builtin_test` with buildifier to simplify future contributions.

Work towards bazelbuild#21833

Closes bazelbuild#21880.

PiperOrigin-RevId: 626315579
Change-Id: Ib7b3d1367586ba7215132d6a9f3711f4ae0a9b09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-Rules-CPP Issues for C++ rules
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants