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

Make rules_testing available for Bazel tests #18182

Closed
wants to merge 1 commit into from

Conversation

fmeum
Copy link
Collaborator

@fmeum fmeum commented Apr 22, 2023

Includes an update of bazel_skylib to 1.3.0 as the (very old) version 1.0.3 wasn't compatible with rules_testing.

Work towards #1920

@fmeum fmeum force-pushed the 1920-add-rules-testing branch 2 times, most recently from e5128b9 to e8c8bd8 Compare April 22, 2023 22:12
@fmeum
Copy link
Collaborator Author

fmeum commented Apr 22, 2023

@rickeylev Could you review this PR?

FYI @oquenchil

@fmeum fmeum marked this pull request as ready for review April 22, 2023 22:31
@github-actions github-actions bot added the awaiting-review PR is awaiting review from an assigned reviewer label Apr 22, 2023
@sgowroji sgowroji added the team-Rules-CPP Issues for C++ rules label Apr 23, 2023
MODULE.bazel Outdated Show resolved Hide resolved
Includes an update of bazel_skylib to the latest version 1.4.1 as the
(very old) version 1.0.3 wasn't compatible with rules_testing.
@rickeylev rickeylev 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 25, 2023
@rickeylev
Copy link
Contributor

/cc @comius as fyi; I unlikely have approval for these code paths internally, so I'll send it to him

@fmeum fmeum deleted the 1920-add-rules-testing branch April 25, 2023 21:05
@sgowroji sgowroji removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Apr 27, 2023
fweikert pushed a commit to fweikert/bazel that referenced this pull request May 25, 2023
Includes an update of bazel_skylib to 1.3.0 as the (very old) version 1.0.3 wasn't compatible with rules_testing.

Work towards bazelbuild#1920

Closes bazelbuild#18182.

PiperOrigin-RevId: 527021012
Change-Id: I80ebff1a1aa656974ad6a1bb1809d8f919a2fa22
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

3 participants