Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Default options on ToDebugString #24498

Merged
merged 2 commits into from
Mar 24, 2021
Merged

Default options on ToDebugString #24498

merged 2 commits into from
Mar 24, 2021

Conversation

carlreid
Copy link
Contributor

I've defaulted the ToDebugString options to be ShortDefault for all the Model/Metadata related and LongDefault for the ChangeTracker related.

Should resolve #24461.

@dnfadmin
Copy link

dnfadmin commented Mar 24, 2021 •
edited
Loading

CLA assistant check
All CLA requirements met.

@ajcvickers ajcvickers merged commit b63d0a6 into dotnet:main Mar 24, 2021
@ajcvickers
Copy link
Contributor

Thanks!

@ajcvickers ajcvickers mentioned this pull request Mar 25, 2021
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@ajcvickers ajcvickers ajcvickers approved these changes

Assignees
No one assigned
Labels
community-contribution
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add a default options setting to ToDebugString
3 participants
@carlreid @dnfadmin @ajcvickers