Skip to content
This repository was archived by the owner on Jan 12, 2024. It is now read-only.

Conversation

@cgranade
Copy link
Contributor

@cgranade cgranade commented Apr 8, 2021

This PR fixes a regression in the documentation generation pipeline that causes some inputs of type Result to be formatted as invalid instead (#949).

@cgranade cgranade requested a review from bettinaheim April 8, 2021 03:36
Copy link
Contributor

@bamarsha bamarsha left a comment

Choose a reason for hiding this comment

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

If only C# had switch exhaustiveness checking...

Co-authored-by: Sarah Marshall <33814365+samarsha@users.noreply.github.com>
@cgranade
Copy link
Contributor Author

cgranade commented Apr 8, 2021

If only C# had switch exhaustiveness checking...

Weirdly, it does, but that feature is undermined by the fact that, in order to support bitflag-style enums, C# enums can have any valid int value. Thus, switching requires a _ pattern, even if the switch is already exhaustive without it, making it really easy to miss cases like this.

Anyway, thanks for reviewing and for catching the bad link!

@cgranade cgranade enabled auto-merge (squash) April 8, 2021 16:07
@cgranade cgranade merged commit cf69050 into main Apr 8, 2021
@cgranade cgranade deleted the cgranade/fix-949 branch April 8, 2021 21:24
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants