Skip to content

process: add type annotations check to CI runs #499

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.

Sign up for GitHub

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

Jump to bottom
Merged
plamut merged 4 commits into from
Sep 27, 2021
Merged

process: add type annotations check to CI runs #499

plamut merged 4 commits into from
Sep 27, 2021

Conversation

@plamut
Copy link
Contributor

@plamut plamut commented Sep 17, 2021 •
edited
Loading

Closes #477.

To verify, examine the CI logs and check that the pytype session has indeed run.

PR checklist:

  • Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
  • Ensure the tests and linter pass
  • Code coverage does not decrease (if any source code was changed)
  • Appropriate docs were updated (if necessary)

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Sep 17, 2021
@product-auto-label product-auto-label bot added the api: pubsub Issues related to the googleapis/python-pubsub API. label Sep 17, 2021
@plamut plamut changed the title process: add type checks to CI runs process: add type annotations check to CI runs Sep 17, 2021
@plamut plamut added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2021
@plamut plamut added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2021
@plamut plamut added the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2021
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Sep 17, 2021
@plamut plamut marked this pull request as ready for review September 17, 2021 12:37
@plamut plamut requested review from a team as code owners September 17, 2021 12:37
@plamut
Copy link
Contributor Author

plamut commented Sep 17, 2021

FWIW, the samples failure is flakiness, will re-run when this is ready to be merged.

@plamut
Copy link
Contributor Author

plamut commented Sep 24, 2021

@pradn @anguillanneuf Friendly ping, this would be a useful addition, thanks!

@plamut plamut mentioned this pull request Sep 24, 2021
Closed
@plamut plamut added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 24, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 24, 2021
@plamut plamut added kokoro:force-run Add this label to force Kokoro to re-run the tests. kokoro:run Add this label to force Kokoro to re-run the tests. labels Sep 24, 2021
@yoshi-kokoro yoshi-kokoro removed kokoro:run Add this label to force Kokoro to re-run the tests. kokoro:force-run Add this label to force Kokoro to re-run the tests. labels Sep 24, 2021
@plamut plamut added the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 24, 2021
@yoshi-kokoro yoshi-kokoro removed the kokoro:force-run Add this label to force Kokoro to re-run the tests. label Sep 24, 2021
@plamut plamut merged commit 58552e7 into googleapis:main Sep 27, 2021
@plamut plamut deleted the iss-477 branch September 27, 2021 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pradn pradn Awaiting requested review from pradn

@anguillanneuf anguillanneuf Awaiting requested review from anguillanneuf

1 more reviewer

@busunkim96 busunkim96 busunkim96 approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

api: pubsub Issues related to the googleapis/python-pubsub API. cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Add type annotation checks to CI

3 participants

@plamut @busunkim96 @yoshi-kokoro