-
Notifications
You must be signed in to change notification settings - Fork 322
Closed
Labels
api: bigqueryIssues related to the googleapis/python-bigquery API.Issues related to the googleapis/python-bigquery API.samplesIssues that are directly related to samples.Issues that are directly related to samples.type: processA process-related concern. May include testing, release, or the like.A process-related concern. May include testing, release, or the like.
Description
This repository has three historical samples locations (see #790 for the issue to migrate them all to the most recent layout, supported by templates), but this issue only refers to the samples in (3). The samples in (1, 2) should remain blackened by the existing mechanism:
docs/snippets.pysamples/*.py+samples/tests/*.pysamples/snippets/*.py,samples/geography/*.py,samples/magics/*.py, and possibly more in the future, see: Clean-upsamples/snippets/requirements.txt#1352.
For this issue, I anticipate the following updates:
- Update the blacken session in noxfile.py to only blacken the samples in legacy directories. That is should be updated to
Line 28 in 34a3f5c
BLACK_PATHS = ("docs", "google", "samples", "tests", "noxfile.py", "setup.py") BLACK_PATHS = ("docs", "google", "samples/*.py", "samples/tests/*.py", "tests", "noxfile.py", "setup.py")or similar. - Update owlbot config to make sure the code samples in
samples/snippets|geography|etcstay blackened. We still want to runnox -s blackenin the root directory but then we also want to runnox -s blackenin each samples directory that has anoxfile.py. See the implementation in python-bigquery-datatransfer for reference: https://github.com/googleapis/python-bigquery-datatransfer/blob/822223fc02b1e2ff6f52b834c51f4bf46924e2e8/owlbot.py#L60-L61
Metadata
Metadata
Assignees
Labels
api: bigqueryIssues related to the googleapis/python-bigquery API.Issues related to the googleapis/python-bigquery API.samplesIssues that are directly related to samples.Issues that are directly related to samples.type: processA process-related concern. May include testing, release, or the like.A process-related concern. May include testing, release, or the like.