Skip to content

Fix for Enum.Format change in behavior between .Net Framework and .Net 5#56261

Merged
stephentoub merged 1 commit into
dotnet:mainfrom
ChrisVicary:issue-56200
Jul 28, 2021
Merged

Fix for Enum.Format change in behavior between .Net Framework and .Net 5#56261
stephentoub merged 1 commit into
dotnet:mainfrom
ChrisVicary:issue-56200

Conversation

@ChrisVicary
Copy link
Copy Markdown
Contributor

Use InternalFormat when formatting an integer
value as an Enum that uses the FlagsAttribute.

Fix #56200

Use InternalFormat when formatting an integer
value as an Enum that uses the FlagsAttribute.

Fix dotnet#56200
@ghost ghost added area-System.Runtime community-contribution Indicates that the PR has been added by a community member labels Jul 24, 2021
@ghost
Copy link
Copy Markdown

ghost commented Jul 24, 2021

Tagging subscribers to this area: @dotnet/area-system-runtime
See info in area-owners.md if you want to be subscribed.

Issue Details

Use InternalFormat when formatting an integer
value as an Enum that uses the FlagsAttribute.

Fix #56200

Author: ChrisVicary
Assignees: -
Labels:

area-System.Runtime

Milestone: -

@dnfadmin
Copy link
Copy Markdown

dnfadmin commented Jul 24, 2021

CLA assistant check
All CLA requirements met.

Copy link
Copy Markdown
Member

@jeffhandley jeffhandley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for submitting this fix!

@jeffhandley jeffhandley self-assigned this Jul 24, 2021
@danmoseley danmoseley changed the title Fix for issue 56200 Fix for Enum.Format change in behavior between .Net Framework and .Net 5 Jul 24, 2021
Copy link
Copy Markdown
Member

@stephentoub stephentoub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks for the fix.

@jeffhandley
Copy link
Copy Markdown
Member

@ChrisVicary It looks like you need to sign our Contributor License Agreement; that will need to be completed before we can merge this PR. Please let us know if you have any questions or concerns about signing it. This comment has the links you need to complete that step.

@ChrisVicary
Copy link
Copy Markdown
Contributor Author

@jeffhandley thanks for letting me know. It shouldn't be a problem, but I'm just waiting for legal approval from my employers. In hindsight I should have done that before submitting the PR, but hopefully it will not take long.

@jeffhandley
Copy link
Copy Markdown
Member

Thanks for the update, @ChrisVicary; no worries at all--totally fine to complete it after your first PR is created.

@danmoseley danmoseley added the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jul 27, 2021
@danmoseley
Copy link
Copy Markdown
Member

Marking no merge meantime. But we do want it :)

@ChrisVicary
Copy link
Copy Markdown
Contributor Author

ChrisVicary commented Jul 28, 2021

@jeffhandley I have been approved, and have signed the CLA, sorry for the delay.

@stephentoub stephentoub removed the NO-MERGE The PR is not ready for merge yet (see discussion for detailed reasons) label Jul 28, 2021
@stephentoub stephentoub merged commit 7ed2959 into dotnet:main Jul 28, 2021
@ChrisVicary ChrisVicary deleted the issue-56200 branch July 28, 2021 14:42
@jeffhandley
Copy link
Copy Markdown
Member

Thank you, @ChrisVicary!!

@danmoseley
Copy link
Copy Markdown
Member

Thanks for your first contribution, @ChrisVicary - we would welcome other contributions if you are interested. There are many issues open to choose from.

@ghost ghost locked as resolved and limited conversation to collaborators Sep 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-System.Runtime community-contribution Indicates that the PR has been added by a community member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Enum.Format change in behavior between .Net Framework and .Net 5

5 participants