Skip to content

[housekeeping] Automated PR to fix formatting errors#1563

Merged
TheCodeTraveler merged 4 commits intomainfrom
housekeeping/fix-codeformatting
Feb 15, 2024
Merged

[housekeeping] Automated PR to fix formatting errors#1563
TheCodeTraveler merged 4 commits intomainfrom
housekeeping/fix-codeformatting

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

Automated PR to fix formatting errors

@github-actions github-actions Bot added the approved This Proposal has been approved and is ready to be added to the Toolkit label Nov 24, 2023
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch from 2fded80 to 8c9d966 Compare November 24, 2023 00:08
@TheCodeTraveler TheCodeTraveler added the do not merge Do not merge this PR label Nov 24, 2023
@TheCodeTraveler TheCodeTraveler removed the approved This Proposal has been approved and is ready to be added to the Toolkit label Nov 24, 2023
@github-actions github-actions Bot added the approved This Proposal has been approved and is ready to be added to the Toolkit label Nov 25, 2023
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch from 8c9d966 to 9177ee6 Compare November 27, 2023 00:09
@jfversluis
Copy link
Copy Markdown
Member

Aggressive bot being aggressive 😄 I guess we want to hold of merging because of those obsolete attributes?

@TheCodeTraveler
Copy link
Copy Markdown
Collaborator

I guess we want to hold of merging because of those obsolete attributes?

Agreed.

It's because dotnet format now adds [Obsolete] attributes when it detects usage of an obsolete API.

We won't be able to get rid of this noise until we upgrade MediaElement away from ExoPlayer, but we're blocked on that: #1511

@bijington
Copy link
Copy Markdown
Contributor

Would it be a bad thing to suppress those warnings just for that part of the namespace until we solve it?

@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch 2 times, most recently from 952ff73 to a1dbd31 Compare November 30, 2023 00:10
@TheCodeTraveler TheCodeTraveler marked this pull request as draft November 30, 2023 18:25
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch from a1dbd31 to bd3f453 Compare December 1, 2023 00:05
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch from bd3f453 to 0d3145b Compare December 8, 2023 00:07
@TheCodeTraveler TheCodeTraveler removed the approved This Proposal has been approved and is ready to be added to the Toolkit label Dec 8, 2023
@github-actions github-actions Bot added the approved This Proposal has been approved and is ready to be added to the Toolkit label Dec 9, 2023
@TheCodeTraveler TheCodeTraveler added blocked and removed approved This Proposal has been approved and is ready to be added to the Toolkit labels Dec 11, 2023
@github-actions github-actions Bot added the approved This Proposal has been approved and is ready to be added to the Toolkit label Dec 12, 2023
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch from 0d3145b to 14af845 Compare January 15, 2024 00:08
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch 2 times, most recently from 3a41b60 to 1d63d57 Compare February 3, 2024 00:08
@github-actions github-actions Bot force-pushed the housekeeping/fix-codeformatting branch from 1d63d57 to 8eff768 Compare February 15, 2024 00:06
@TheCodeTraveler TheCodeTraveler marked this pull request as ready for review February 15, 2024 21:31
Copy link
Copy Markdown
Collaborator

@TheCodeTraveler TheCodeTraveler left a comment

Choose a reason for hiding this comment

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

dotnet format no longer adds [ObsoleteAttribute]

@TheCodeTraveler TheCodeTraveler merged commit 97440cc into main Feb 15, 2024
@TheCodeTraveler TheCodeTraveler deleted the housekeeping/fix-codeformatting branch February 15, 2024 21:32
@github-actions github-actions Bot locked and limited conversation to collaborators Nov 20, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

approved This Proposal has been approved and is ready to be added to the Toolkit blocked do not merge Do not merge this PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants