Skip to content
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.

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

Regenerate .pyi files, sort diffs on proto cksum #11313

Merged
merged 1 commit into from Nov 10, 2022

Conversation

iwahbe
Copy link
Member

@iwahbe iwahbe commented Nov 9, 2022

Regenerate the .pyi files as left from #11095.

It's unclear to me why this didn't happen in the initial PR. The only thing I can think of is that .checksum.txt was manually edited.

@iwahbe iwahbe self-assigned this Nov 9, 2022
@pulumi-bot
Copy link
Contributor

pulumi-bot commented Nov 9, 2022

Changelog

[uncommitted] (2022-11-10)

Copy link
Member

@Frassle Frassle left a comment

Choose a reason for hiding this comment

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

We should probably have a CI check that make-proto doesn't make diff changes

@iwahbe iwahbe added the impact/no-changelog-required This issue doesn't require a CHANGELOG update label Nov 9, 2022
@iwahbe
Copy link
Member Author

iwahbe commented Nov 9, 2022

We should probably have a CI check that make-proto doesn't make diff changes

We do, but I think its broken. It let #11095 through but is complaining on this PR.

The test was fine, the issue cam up because of a mac specific weirdness with find. Changing find proto/ ... to find proto ... fixed that. I'm still unsure how some of the proto/.checksum.txt fields got updated without the full regen.

@iwahbe iwahbe added this to the 0.80 milestone Nov 9, 2022
@iwahbe iwahbe force-pushed the iwahbe/regenerate-proto-for-python branch 3 times, most recently from 8e101d5 to d0982c6 Compare November 10, 2022 00:37
@iwahbe iwahbe force-pushed the iwahbe/regenerate-proto-for-python branch from d0982c6 to eecbe29 Compare November 10, 2022 00:49
@iwahbe
Copy link
Member Author

iwahbe commented Nov 10, 2022

bors r+

@bors
Copy link
Contributor

bors bot commented Nov 10, 2022

Build succeeded:

@bors bors bot merged commit 42cb2c8 into master Nov 10, 2022
@pulumi-bot pulumi-bot deleted the iwahbe/regenerate-proto-for-python branch November 10, 2022 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
impact/no-changelog-required This issue doesn't require a CHANGELOG update language/python
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants