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

test(pip_parse): add integration tests for macros in requirements.bzl #1205

Closed

Conversation

aignas
Copy link
Collaborator

@aignas aignas commented May 4, 2023

The PR #1190 fixed incompatible_generate_aliases breaking the
requirement macro under bzlmod. This PR is a followup in
establishing the test infrastructure using the classic bazel only for
now.

Work towards #958.

Previously we would depend on the value of .bazelrc and this
change ensures that we are explicitly enable bzlmod via CLI
args.
@aignas
Copy link
Collaborator Author

aignas commented May 4, 2023

After #1155, #1176, #1204, we can add an integration test target which
would exercise the same BUILD.bazel files under bzlmod and we could
potentially remove build_test definitions in the bzlmod example added
in #1176.

@aignas aignas force-pushed the test/non-example-pip-parse-tests branch from ac252f1 to d8429df Compare May 4, 2023 06:29
rickeylev pushed a commit that referenced this pull request May 4, 2023
…d in .bazelrc (#1207)

This correctly handles the integration tests and examples that are a
part of
the `rules_python` workspace and should not be included in the deleted
packages
list.

This brings the changes made to the `.bazelrc` very close to what is in
`main`,
but I would like to update this later once #1155 and #1205 are merged.

Fixes #919.
aignas added 3 commits May 5, 2023 20:01
The PR bazelbuild#1190 fixed incompatible_generate_aliases breaking the
`requirement` macro under `bzlmod`. This PR is a followup in
establishing the test infrastructure using the classic `bazel` only for
now.

What is more, update the deleted packages with the fixed version of the
script from bazelbuild#1207
@aignas aignas force-pushed the test/non-example-pip-parse-tests branch from d8429df to 521486d Compare May 5, 2023 11:15
@chrislovecnm
Copy link
Collaborator

Ping

@aignas aignas closed this Jul 10, 2023
@aignas aignas deleted the test/non-example-pip-parse-tests branch July 10, 2023 01:26
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