Skip to content
This repository was archived by the owner on Sep 5, 2023. It is now read-only.

chore: generate python samples templates in owlbot.py #99

Merged
merged 2 commits into from
Aug 18, 2021

Conversation

busunkim96
Copy link
Contributor

Generate python samples templates in owlbot.py

@busunkim96 busunkim96 requested a review from a team as a code owner August 17, 2021 20:54
@busunkim96 busunkim96 requested a review from parthea August 17, 2021 20:54
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Aug 17, 2021
@product-auto-label product-auto-label bot added api: cloudfunctions Issues related to the googleapis/python-functions API. samples Issues that are directly related to samples. labels Aug 17, 2021
Copy link
Contributor

@parthea parthea left a comment

Choose a reason for hiding this comment

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

One minor observation, otherwise LGTM. There were no templated sample files added. Is that expected?

Comment on lines +35 to 38
python.py_samples(skip_readmes=True)
s.move(
templated_files, excludes=[".coveragerc"]
) # the microgenerator has a good coveragerc file
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
python.py_samples(skip_readmes=True)
s.move(
templated_files, excludes=[".coveragerc"]
) # the microgenerator has a good coveragerc file
s.move(
templated_files, excludes=[".coveragerc"]
) # the microgenerator has a good coveragerc file
python.py_samples(skip_readmes=True)

Copy link
Contributor

Choose a reason for hiding this comment

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

One minor observation, otherwise LGTM. There were no templated sample files added. Is that expected?

@parthea parthea added the automerge Merge the pull request once unit tests and other checks pass. label Aug 18, 2021
@gcf-merge-on-green gcf-merge-on-green bot merged commit d96ff05 into master Aug 18, 2021
@gcf-merge-on-green gcf-merge-on-green bot deleted the generate-samples-noxfile branch August 18, 2021 13:58
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Aug 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
api: cloudfunctions Issues related to the googleapis/python-functions API. cla: yes This human has signed the Contributor License Agreement. samples Issues that are directly related to samples.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants