Skip to content

chore: enable context-aware commits for one API - #5085

Merged
dazuma merged 1 commit into
masterfrom
context-aware-commits
Mar 19, 2020
Merged

dazuma merged 1 commit into
masterfrom
context-aware-commits

Conversation

@SurferJeffAtGoogle

Copy link
Copy Markdown
Contributor

No description provided.

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 19, 2020

@dazuma dazuma left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

A python global seems like an odd way to configure this. I'll trust that it's intentional for now because I don't have a way of testing this locally.

@SurferJeffAtGoogle

Copy link
Copy Markdown
Contributor Author

A python global seems like an odd way to configure this. I'll trust that it's intentional for now because I don't have a way of testing this locally.

Thanks for the quick review. Yes, it's intentional. Here are some other synth.py's configured the same way: https://github.com/search?q=AUTOSYNTH_MULTIPLE_COMMITS&type=Code

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.

3 participants