Skip to content

Fix unhandled ResultStatus.NoContent in MinimalApiResultExtensions.ToMinimalApiResult#192

Merged
ardalis merged 1 commit intoardalis:mainfrom
inghamc:issue191_ToMinimalApiResult_NoContent
Jun 18, 2024
Merged

Fix unhandled ResultStatus.NoContent in MinimalApiResultExtensions.ToMinimalApiResult#192
ardalis merged 1 commit intoardalis:mainfrom
inghamc:issue191_ToMinimalApiResult_NoContent

Conversation

@inghamc
Copy link
Contributor

@inghamc inghamc commented Jun 14, 2024

Added an xUnit test to avoid future unhandled ResultStatus values
Fixes #191

@ardalis
Copy link
Owner

ardalis commented Jun 18, 2024

Wow this test output is tough to read... https://github.com/ardalis/Result/actions/runs/9508873176/job/26388508329?pr=192

I'm going to go ahead and merge this and see what's failing locally...

@ardalis ardalis merged commit bb930b3 into ardalis:main Jun 18, 2024
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.

MinimalApiResultExtensions.ToMinimalApiResult is missing case for ResultStatus.NoContent

2 participants