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

feat: Introduce compatibility with native namespace packages #1720

Closed
wants to merge 2 commits into from

Conversation

parthea
Copy link
Contributor

@parthea parthea commented Nov 13, 2023

@product-auto-label product-auto-label bot added size: m Pull request size is medium. api: bigquery Issues related to the googleapis/python-bigquery API. labels Nov 13, 2023
@parthea parthea force-pushed the migrate-to-native-namespace-packages branch 3 times, most recently from 695affd to fbeb2b0 Compare November 13, 2023 17:58
@parthea parthea added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Nov 13, 2023
@parthea
Copy link
Contributor Author

parthea commented Nov 13, 2023

Adding do not merge as there is an upcoming release freeze. This should be merged after the release freeze ends on November 29th.

@parthea parthea force-pushed the migrate-to-native-namespace-packages branch from fbeb2b0 to c9a0753 Compare November 13, 2023 20:24
@parthea parthea self-assigned this Nov 13, 2023
@parthea parthea force-pushed the migrate-to-native-namespace-packages branch 2 times, most recently from 336a9dc to e69ce81 Compare November 14, 2023 10:48
@parthea parthea force-pushed the migrate-to-native-namespace-packages branch from e69ce81 to df642b0 Compare November 14, 2023 10:49
@parthea parthea marked this pull request as ready for review November 14, 2023 15:05
@parthea parthea requested review from a team as code owners November 14, 2023 15:05
@parthea parthea requested a review from mrfaizal November 14, 2023 15:05
Copy link
Contributor

@kiraksi kiraksi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

google.joinpath("othermod.py").write_text("")
env = dict(os.environ, PYTHONPATH=str(tmp_path))
cmd = [sys.executable, "-m", "google.othermod"]
subprocess.check_call(cmd, env=env)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Were you able to run this test with subprocess without specifying to do it in the shell? It wasn't able to find the tmp path without me specifying to run in the shell, which is the small edit I did in the other PR for this

@parthea
Copy link
Contributor Author

parthea commented Nov 21, 2023

Superseded by #1726

@parthea parthea closed this Nov 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
api: bigquery Issues related to the googleapis/python-bigquery API. do not merge Indicates a pull request not ready for merge, due to either quality or timing. size: m Pull request size is medium.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

use native support for namespace packages
2 participants