Skip to content

Implement allow_alias for enums#607

Merged
mcy merged 2 commits into
mainfrom
mcy/allow-alias
Oct 17, 2025
Merged

Implement allow_alias for enums#607
mcy merged 2 commits into
mainfrom
mcy/allow-alias

Conversation

@mcy

@mcy mcy commented Oct 15, 2025

Copy link
Copy Markdown
Member

Protobuf allows enum values to alias if the allow_alias option is set. This PR:

  1. Implements this behavior.
  2. Generates a required error when allow_alias is set but no aliasing values exist.
  3. Fixes some bugs around zero-valued enum values not having uniqueness/reserved constraints checked correctly.

@mcy
mcy enabled auto-merge (squash) October 17, 2025 18:43
@mcy
mcy merged commit 04b6b53 into main Oct 17, 2025
6 checks passed
@mcy
mcy deleted the mcy/allow-alias branch October 17, 2025 18:48
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