Skip to content

Added CriticalError to ResultStatuses (#140) - #141

Merged
ardalis merged 2 commits into
ardalis:mainfrom
EnochMtzR:CriticalErrorResult-EnochMtzR
Sep 12, 2023
Merged

ardalis merged 2 commits into
ardalis:mainfrom
EnochMtzR:CriticalErrorResult-EnochMtzR

Conversation

@EnochMtzR

Copy link
Copy Markdown
Contributor

This is my take on adding the Critical Error status

@EnochMtzR EnochMtzR mentioned this pull request Jun 16, 2023
@EnochMtzR EnochMtzR linked an issue Jun 18, 2023 that may be closed by this pull request
@ardalis

ardalis commented Jun 28, 2023

Copy link
Copy Markdown
Owner

Can you update the tests?

@ardalis

ardalis commented Sep 12, 2023

Copy link
Copy Markdown
Owner

Hmm CI build is still failing even though it says tests passed and no errors. I'll merge it and see if there's anything I need to fix. Thanks!

@ardalis
ardalis merged commit 8e93188 into ardalis:main Sep 12, 2023
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.

Critical ErrorResult

2 participants