This repository was archived by the owner on Mar 6, 2026. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 325
standardize samples directory layout #790
Copy link
Copy link
Labels
api: bigqueryIssues related to the googleapis/python-bigquery API.Issues related to the googleapis/python-bigquery API.priority: p3Desirable enhancement or fix. May not be included in next release.Desirable enhancement or fix. May not be included in next release.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.
Metadata
Metadata
Assignees
Labels
api: bigqueryIssues related to the googleapis/python-bigquery API.Issues related to the googleapis/python-bigquery API.priority: p3Desirable enhancement or fix. May not be included in next release.Desirable enhancement or fix. May not be included in next release.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.
Type
Fields
Give feedbackNo fields configured for issues without a type.
We have an odd mix of sample locations:
docs/snippets.py-- the original location, not very copy-pasteable / browsablesamples/*.py, but the test drivers are insamples/tests/samples/snippets/*.pysamples/geography/*.pyTo align with the infrastructure in https://github.com/googleapis/synthtool/tree/master/synthtool/gcp/templates/python_samples, the goal is to have:
samples/snippets/-- these samples just require thegoogle-cloud-bigquerylibrarysamples/geography/-- require extra dependencies for working with geography datasample/pandas/-- require extra dependencies for working with pandas dataframesTo avoid breaking cloud.google.com docs, this needs to be done in several steps:
Towards: #366 (use templated noxfile) and
#765 (blacken samples using templated noxfile(s))