Skip to content

Conversation

@SimonCropp
Copy link
Contributor

  • leverage CallerArgumentExpressionAttribute
  • there were two exception handling helpers. Error and Utils. Consolidated on Utils. Happy to rename if u want
  • the string format and CultureInfo code in Error was redundant, since the only place it was used in was not doing any culture sensitive formats.

@SimonCropp
Copy link
Contributor Author

conflict resolved

baywet
baywet previously approved these changes Oct 5, 2023
@baywet baywet enabled auto-merge October 5, 2023 12:37
@baywet
Copy link
Member

baywet commented Oct 5, 2023

@SimonCropp this one is conflicting again, would you mind rebasing please?

@SimonCropp
Copy link
Contributor Author

@baywet done

@baywet baywet merged commit eb97519 into microsoft:vnext Oct 5, 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.

2 participants