Skip to content

Conversation

@chalmerlowe
Copy link
Collaborator

Adds a noxfile.py to assist with testing the microgenerator and performing tasks such as blacken and lint.

@chalmerlowe chalmerlowe requested review from a team as code owners September 11, 2025 18:49
@chalmerlowe chalmerlowe requested review from suzmue and removed request for a team September 11, 2025 18:49
@product-auto-label product-auto-label bot added the size: l Pull request size is large. label Sep 11, 2025
@product-auto-label product-auto-label bot added the api: bigquery Issues related to the googleapis/python-bigquery API. label Sep 11, 2025
@chalmerlowe chalmerlowe added this to the µgen PoC milestone Sep 12, 2025
@chalmerlowe chalmerlowe changed the title Chore: microgen - add noxfile for the microgenerator chore: microgen - add noxfile for the microgenerator Sep 15, 2025
Copy link
Contributor

@leahecole leahecole left a comment

Choose a reason for hiding this comment

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

Is this noxfile derived from other noxfiles we use? If so, is there anything we need to do to make sure this is set up to be kept up to date by our other automation? If not, I would just ask we open an internal tracking bug to get to that at some point

@chalmerlowe
Copy link
Collaborator Author

chalmerlowe commented Sep 15, 2025

Is this noxfile derived from other noxfiles we use? If so, is there anything we need to do to make sure this is set up to be kept up to date by our other automation? If not, I would just ask we open an internal tracking bug to get to that at some point

The file is based on some of the noxfiles we already use.

We don't yet have a plan in place on where to store this microgenerator or how to ensure that it gets run on the GAPIC generated code.

@Linchin has the task to look into much of that. I believe that handling the noxfile is a part of that overall task.

I am tracking some of the requests from this PR series (including this request) to keep them all in place for once we get the alpha fully merged and can start looking at the next steps.

Base automatically changed from feat-adds-config-file to autogen September 15, 2025 15:46
@chalmerlowe chalmerlowe merged commit 67f3d07 into autogen Sep 15, 2025
14 of 25 checks passed
@chalmerlowe chalmerlowe deleted the chore-add-microgen-noxfile branch September 15, 2025 16:56
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 API. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants