Skip to content

Include ValidationFailure.ErrorCode in ExecutionError.Code.#440

Merged
SimonCropp merged 1 commit intoSimonCropp:masterfrom
danielcweber:IncludeErrorCode
Feb 13, 2021
Merged

Include ValidationFailure.ErrorCode in ExecutionError.Code.#440
SimonCropp merged 1 commit intoSimonCropp:masterfrom
danielcweber:IncludeErrorCode

Conversation

@danielcweber
Copy link
Contributor

There might be a rationale for not including these (type names leaking?), but as far as I can see, it's not configurable so it looks more like an omission.

@SimonCropp
Copy link
Owner

it looks more like an omission.

yep. And thanks. will merge and deploy now

@SimonCropp SimonCropp added this to the 5.2.0 milestone Feb 13, 2021
@SimonCropp SimonCropp merged commit 58f59dc into SimonCropp:master Feb 13, 2021
@danielcweber danielcweber deleted the IncludeErrorCode branch February 13, 2021 19:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants