Skip to content

Bump protobuf to 3.20.1 #8609

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

Merged
merged 4 commits into from
Aug 25, 2022
Merged

Conversation

nipunn1313
Copy link
Contributor

There are even newer versions available (4.*), but I had found a bug
in it which prevented it from working with mypy-protobuf. For now,
sticking to just 3.20.1 until that stuff is resolved.

@nipunn1313 nipunn1313 marked this pull request as draft August 25, 2022 11:18
@nipunn1313
Copy link
Contributor Author

(converting to draft so I can fix stubtest)

@github-actions

This comment has been minimized.

There are even newer versions available (4.*), but I had found a bug
in it which prevented it from working with mypy-protobuf. For now,
sticking to just 3.20.1
@nipunn1313 nipunn1313 force-pushed the bump_protobuf_3_20_only branch from d708b54 to 3908808 Compare August 25, 2022 11:44
@nipunn1313 nipunn1313 marked this pull request as ready for review August 25, 2022 11:45
@nipunn1313
Copy link
Contributor Author

I pushed some new code - but the github CI hasn't seemed to rerun yet - not sure what the hold up is. Poking around.

@github-actions

This comment has been minimized.

1 similar comment
@github-actions

This comment has been minimized.

@AlexWaygood
Copy link
Member

I pushed some new code - but the github CI hasn't seemed to rerun yet - not sure what the hold up is. Poking around.

I think it did re-run? https://github.com/python/typeshed/runs/8015005080?check_suite_focus=true (the later stubtest run) has different errors to the earlier stubtest run (https://github.com/python/typeshed/runs/8014591866?check_suite_focus=true) 🙂

@nipunn1313
Copy link
Contributor Author

it just took a while to run. I think there's some limitation where it waits for the previous run to fully complete before starting another run.

@github-actions

This comment has been minimized.

@github-actions
Copy link
Contributor

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

@nipunn1313
Copy link
Contributor Author

Ok looks like it completed successfully!

Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Is it worth bumping the mypy-protobuf version being used here?

@@ -11,7 +11,7 @@ set -ex -o pipefail
# followed by committing the changes to typeshed
#
# Update these two variables when rerunning script
PROTOBUF_VERSION=3.19.3
PROTOBUF_VERSION=3.20.1
MYPY_PROTOBUF_VERSION=v3.2.0
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
MYPY_PROTOBUF_VERSION=v3.2.0
MYPY_PROTOBUF_VERSION=v3.3.0

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, but I'd like to do it separately - because it ends up having a pretty different set of changes.

I originally was preparing a PR that did both all at once, but it ends up with a bunch of different changes since it changes the code-generator utility.

Would prefer to keep the two PRs isolated.

Copy link
Member

Choose a reason for hiding this comment

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

Sure, makes sense!

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was also planning to test the changes with 3.3.0 and if things don't work out perfectly, I may have to release 3.3.1 (or 3.4.0) to get typeshed to work.

@@ -1,2 +1,2 @@
version = "3.19.*"
version = "3.20.*"
extra_description = "Generated with aid from mypy-protobuf v3.2.0"
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
extra_description = "Generated with aid from mypy-protobuf v3.2.0"
extra_description = "Generated with aid from mypy-protobuf v3.3.0"

Copy link
Member

@AlexWaygood AlexWaygood left a comment

Choose a reason for hiding this comment

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

Haven't reviewed this in depth, but nothing sticks out, and you're the expert when it comes to these stubs!

@AlexWaygood AlexWaygood merged commit c17c0d5 into python:master Aug 25, 2022
@nipunn1313
Copy link
Contributor Author

sweet! Will send you another one updating mypy-protobuf too!

@nipunn1313 nipunn1313 deleted the bump_protobuf_3_20_only branch October 20, 2022 06:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants