Skip to content
This repository has been archived by the owner on Jul 31, 2024. It is now read-only.

Unify empty string #2730

Merged
merged 2 commits into from
Nov 4, 2018
Merged

Unify empty string #2730

merged 2 commits into from
Nov 4, 2018

Conversation

milbrandt
Copy link
Contributor

What issue does this PR address?

  • unify empty string ("" / string.Empty) in C#
    -nameof for ArgumentExceptions

Does this PR introduce a breaking change?
no

Please check if the PR fulfills these requirements

  • The commit follows our guidelines
  • Unit Tests for the changes have been added (for bug fixes / features)

Other information:

@dnfclas
Copy link

dnfclas commented Oct 18, 2018

CLA assistant check
All CLA requirements met.

@brockallen brockallen merged commit 619760e into IdentityServer:dev Nov 4, 2018
@brockallen
Copy link
Member

Thanks.

@brockallen brockallen added this to the 2.3 milestone Nov 4, 2018
@milbrandt milbrandt deleted the emptyString branch November 4, 2018 18:13
@lock
Copy link

lock bot commented Jan 10, 2020

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 10, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants