Skip to content

Add handling for protobuf==7#429

Merged
stefanvanburen merged 3 commits intomainfrom
svanburen/protobuf-7
Mar 2, 2026
Merged

Add handling for protobuf==7#429
stefanvanburen merged 3 commits intomainfrom
svanburen/protobuf-7

Conversation

@stefanvanburen
Copy link
Member

protobuf v7 was released last week, which dropped support for some of the APIs that we use for checking field types. This fixes those accesses so that we can still support all the way back to v5.

Fixes #410.

Ref: https://pypi.org/project/protobuf/
Ref: https://github.com/protocolbuffers/protobuf/releases/tag/v34.0

`protobuf` v7 was released last week, which dropped support for some of
the APIs that we use for checking field types. This fixes those accesses
so that we can still support all the way back to v5.

Fixes #410.

Ref: https://pypi.org/project/protobuf/
Ref: https://github.com/protocolbuffers/protobuf/releases/tag/v34.0
@stefanvanburen stefanvanburen marked this pull request as ready for review March 2, 2026 14:31
@AdrienVannson
Copy link
Contributor

Do we actually want to keep supporting v5?

According to the documentation, it is maintenance-only (https://protobuf.dev/support/version-support) and support will end at the end of the month (https://protobuf.dev/support/version-support/#python)

@stefanvanburen
Copy link
Member Author

@AdrienVannson good question — for now, I think we ought to leave around support since it's trivial on our end. Last time I looked, a lot of OSS libraries and users were still using 5.x. I do think we could probably reconsider that at EOM.

I'd like to release this, and then maybe we can drop support for the next release?

@AdrienVannson
Copy link
Contributor

I see. Apart from this, this looks good

@stefanvanburen stefanvanburen merged commit f85bf45 into main Mar 2, 2026
11 checks passed
@stefanvanburen stefanvanburen deleted the svanburen/protobuf-7 branch March 2, 2026 14:59
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.

Using deprecated Protobuf APIs

2 participants