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

docs: omit mention of Python 2.7 in 'CONTRIBUTING.rst' #191

Conversation

gcf-owl-bot[bot]
Copy link
Contributor

@gcf-owl-bot gcf-owl-bot bot commented Jun 18, 2021

Source-Link: googleapis/synthtool@b91f129
Post-Processor: gcr.io/repo-automation-bots/owlbot-python:latest@sha256:b6169fc6a5207b11800a7c002d0c5c2bc6d82697185ca12e666f44031468cfcd

This PR also contains the following maintenance updates:

chore: add kokoro 3.9 config templates
chore: update precommit hook pre-commit/pre-commit-hooks to v4
chore(python): remove hardcoded versions in CONTRIBUTING.rst

@gcf-owl-bot gcf-owl-bot bot requested a review from a team as a code owner June 18, 2021 19:52
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Jun 18, 2021
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery-sqlalchemy API. label Jun 18, 2021
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jun 18, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 18, 2021
CONTRIBUTING.rst Outdated Show resolved Hide resolved
@parthea parthea force-pushed the owl-bot-update-lock-b6169fc6a5207b11800a7c002d0c5c2bc6d82697185ca12e666f44031468cfcd branch from fcdf5e3 to 07e4ac6 Compare June 18, 2021 21:39
@trusted-contributions-gcf trusted-contributions-gcf bot added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 18, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 18, 2021
parthea
parthea previously approved these changes Jun 18, 2021
@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Jun 19, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 19, 2021
@parthea parthea added do not merge Indicates a pull request not ready for merge, due to either quality or timing. and removed automerge Merge the pull request once unit tests and other checks pass. labels Jun 19, 2021
@parthea
Copy link
Contributor

parthea commented Jun 22, 2021

Blocked by googleapis/synthtool#1132

@parthea parthea changed the title docs: omit mention of Python 2.7 in 'CONTRIBUTING.rst' (#1127) docs: omit mention of Python 2.7 in 'CONTRIBUTING.rst' Jun 22, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added kokoro:force-run Add this label to force Kokoro to re-run the tests. owlbot:run Add this label to trigger the Owlbot post processor. labels Jun 22, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 22, 2021
@parthea parthea added owlbot:run Add this label to trigger the Owlbot post processor. and removed owlbot:run Add this label to trigger the Owlbot post processor. labels Jun 22, 2021
@trusted-contributions-gcf trusted-contributions-gcf bot added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 22, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 22, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Jun 22, 2021
@parthea
Copy link
Contributor

parthea commented Jun 22, 2021

@jimfulton PTAL

@parthea parthea removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jun 22, 2021
@parthea parthea dismissed their stale review June 22, 2021 18:28

Need review from @googleapis/api-bigquery

Copy link
Contributor

@jimfulton jimfulton left a comment

Choose a reason for hiding this comment

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

I don't know what all the config file changes do :)

But wrt CONTRIBUTING.rst, I suggested some simplification that are more user friendly and version independent.

Comment on lines +72 to +75
$ nox -s unit-3.6
$ nox -s unit-3.7
$ nox -s unit-3.8
$ ...
$ nox -s unit-3.9
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be simplified to:

$ nox -s unit

Comment on lines 148 to +149
$ nox -s system-3.8
$ nox -s system-2.7
$ nox -s system-3.9
Copy link
Contributor

Choose a reason for hiding this comment

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

This should be simplified to:

nox -s system

@parthea
Copy link
Contributor

parthea commented Jun 22, 2021

Blocked by googleapis/synthtool#1135

@parthea parthea requested a review from tswast June 22, 2021 19:07
@parthea
Copy link
Contributor

parthea commented Jun 22, 2021

Added @tswast as reviewer for the kokoro changes.

@parthea
Copy link
Contributor

parthea commented Jun 22, 2021

Thanks for reviewing, @jimfulton ! I'll request another review once the templates are ready.

gcf-merge-on-green bot pushed a commit to googleapis/synthtool that referenced this pull request Jun 25, 2021
@parthea
Copy link
Contributor

parthea commented Jun 25, 2021

Superseded by #195

@parthea parthea closed this Jun 25, 2021
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-sqlalchemy API. cla: yes This human has signed the Contributor License Agreement. owl-bot-update-lock
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants