Skip to content

chore(python): modify templated noxfile to support non-cloud APIs#1263

Merged
parthea merged 1 commit intomasterfrom
add-support-for-non-cloud-in-noxfile
Oct 21, 2021
Merged

chore(python): modify templated noxfile to support non-cloud APIs#1263
parthea merged 1 commit intomasterfrom
add-support-for-non-cloud-in-noxfile

Conversation

@parthea
Copy link
Copy Markdown
Contributor

@parthea parthea commented Oct 21, 2021

Currently the owlbot.py file for non-cloud APIs need to have replacements in owlbot.py in order to use the templated noxfile. See the example from python-analytics-admin here which has:

s.replace("noxfile.py",
'''["']--cov=google/cloud["'],''',
'''"--cov=google.analytics",''')

This PR eliminates the need for this replacement in owlbot.py. I tested this change in googleapis/python-orchestration-airflow#41, and googleapis/python-analytics-admin#157 and the Kokoro build passed in both.

@parthea parthea requested a review from a team October 21, 2021 00:37
@google-cla google-cla Bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 21, 2021
@parthea parthea requested a review from busunkim96 October 21, 2021 01:17
@parthea parthea merged commit 76d5fec into master Oct 21, 2021
@parthea parthea deleted the add-support-for-non-cloud-in-noxfile branch October 21, 2021 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants