Title: blacken samples using templated noxfile · Issue #765 · googleapis/python-bigquery · GitHub
Open Graph Title: blacken samples using templated noxfile · Issue #765 · googleapis/python-bigquery
X Title: blacken samples using templated noxfile · Issue #765 · googleapis/python-bigquery
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 ...
Open Graph 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 (...
X 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 (...
Opengraph URL: https://github.com/googleapis/python-bigquery/issues/765
X: @github
Domain: github.com
{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"blacken samples using templated noxfile","articleBody":"This repository has three historical samples locations (see https://github.com/googleapis/python-bigquery/issues/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:\r\n\r\n1. `docs/snippets.py`\r\n2. `samples/*.py` + `samples/tests/*.py`\r\n3. `samples/snippets/*.py`, `samples/geography/*.py`, `samples/magics/*.py`, and possibly more in the future, see: https://github.com/googleapis/python-bigquery/issues/1352.\r\n\r\nFor this issue, I anticipate the following updates:\r\n\r\n1. Update the blacken session in noxfile.py to only blacken the samples in legacy directories. That is https://github.com/googleapis/python-bigquery/blob/34a3f5cf34d4a08889fe1407f4ad6ce3c9d93838/noxfile.py#L28 should be updated to `BLACK_PATHS = (\"docs\", \"google\", \"samples/*.py\", \"samples/tests/*.py\", \"tests\", \"noxfile.py\", \"setup.py\")` or similar.\r\n2. Update owlbot config to make sure the code samples in `samples/snippets|geography|etc` stay blackened. We still want to run `nox -s blacken` in the root directory but then we also want to run `nox -s blacken` in each samples directory that has a `noxfile.py`. See the implementation in python-bigquery-datatransfer for reference: https://github.com/googleapis/python-bigquery-datatransfer/blob/822223fc02b1e2ff6f52b834c51f4bf46924e2e8/owlbot.py#L60-L61","author":{"url":"https://github.com/tswast","@type":"Person","name":"tswast"},"datePublished":"2021-07-14T20:27:25.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/765/python-bigquery/issues/765"}
| route-pattern | /_view_fragments/issues/show/:user_id/:repository/:id/issue_layout(.:format) |
| route-controller | voltron_issues_fragments |
| route-action | issue_layout |
| fetch-nonce | v2:5bc82786-aa15-db4c-9e56-93a654a2ceba |
| current-catalog-service-hash | 81bb79d38c15960b92d99bca9288a9108c7a47b18f2423d0f6438c5b7bcd2114 |
| request-id | D42A:25572C:221060C:2EA5498:6A4EC7C6 |
| html-safe-nonce | 57f1e3cf87d60760eea474ae81ab67a8a04aa1cbb16426f987fee89191a6fa4d |
| visitor-payload | eyJyZWZlcnJlciI6IiIsInJlcXVlc3RfaWQiOiJENDJBOjI1NTcyQzoyMjEwNjBDOjJFQTU0OTg6NkE0RUM3QzYiLCJ2aXNpdG9yX2lkIjoiODQxNTE1ODExMDkyMjkxNzgzMCIsInJlZ2lvbl9lZGdlIjoiaWFkIiwicmVnaW9uX3JlbmRlciI6ImlhZCJ9 |
| visitor-hmac | 8454c650a13b26c146fd72e60ea6bd012ab3a1e9a32878201c303172000d7386 |
| hovercard-subject-tag | issue:944771412 |
| github-keyboard-shortcuts | repository,issues,copilot |
| google-site-verification | Apib7-x98H0j5cPqHWwSMm6dNU4GmODRoqxLiDzdx9I |
| octolytics-url | https://collector.github.com/github/collect |
| analytics-location | / |
| fb:app_id | 1401488693436528 |
| apple-itunes-app | app-id=1477376905, app-argument=https://github.com/_view_fragments/issues/show/googleapis/python-bigquery/765/issue_layout |
| twitter:image | https://opengraph.githubassets.com/09d8845a5187dd6b58f6e99fc5449f42fb9ba29c288e03140a6300214add2223/googleapis/python-bigquery/issues/765 |
| twitter:card | summary_large_image |
| og:image | https://opengraph.githubassets.com/09d8845a5187dd6b58f6e99fc5449f42fb9ba29c288e03140a6300214add2223/googleapis/python-bigquery/issues/765 |
| og:image:alt | 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 (... |
| og:image:width | 1200 |
| og:image:height | 600 |
| og:site_name | GitHub |
| og:type | object |
| og:author:username | tswast |
| hostname | github.com |
| expected-hostname | github.com |
| None | 41b6ab3ba6d20a71766ac245b5a4a94c6fc672a9cd4da7d44c1b33ab8bf6a21c |
| turbo-cache-control | no-preview |
| go-import | github.com/googleapis/python-bigquery git https://github.com/googleapis/python-bigquery.git |
| octolytics-dimension-user_id | 16785467 |
| octolytics-dimension-user_login | googleapis |
| octolytics-dimension-repository_id | 226992475 |
| octolytics-dimension-repository_nwo | googleapis/python-bigquery |
| octolytics-dimension-repository_public | true |
| octolytics-dimension-repository_is_fork | false |
| octolytics-dimension-repository_network_root_id | 226992475 |
| octolytics-dimension-repository_network_root_nwo | googleapis/python-bigquery |
| turbo-body-classes | logged-out env-production page-responsive |
| disable-turbo | false |
| browser-stats-url | https://api.github.com/_private/browser/stats |
| browser-errors-url | https://api.github.com/_private/browser/errors |
| release | e6a744804e8e70f97b4d5a18a94dcc63db22f97a |
| ui-target | canary-2 |
| theme-color | #1e2327 |
| color-scheme | light dark |
Links:
Viewport: width=device-width