Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Fix IDE1006 warning - #2112

#2112
Merged
martincostello merged 2 commits into
App-vNext:mainApp-vNext/Polly:mainfrom
iamdmitrij:IDE1006-fixiamdmitrij/Polly:IDE1006-fixCopy head branch name to clipboard
May 16, 2024
Merged

Fix IDE1006 warning#2112
martincostello merged 2 commits into
App-vNext:mainApp-vNext/Polly:mainfrom
iamdmitrij:IDE1006-fixiamdmitrij/Polly:IDE1006-fixCopy head branch name to clipboard

Conversation

@iamdmitrij

@iamdmitrij iamdmitrij commented May 16, 2024

Copy link
Copy Markdown
Contributor

Pull Request

The issue or feature being addressed

#1290

Details on the issue fix or feature implementation

  • Suppress IDE1006 in the code or fix the warning

Similar concept to #2111, if warning related to public API - suppress in the code, otherwise fix.

Confirm the following

  • I started this PR by branching from the head of the default branch
  • I have targeted the PR to merge into the default branch
  • I have included unit tests for the issue/feature
  • I have successfully run a local build

Fix public API file

Remove breaking changes from PublicAPI.Shipped.txt

Move suppressions above summary comments

Remove S3872
@martincostello
martincostello enabled auto-merge (squash) May 16, 2024 06:13
@martincostello
martincostello merged commit fae7486 into App-vNext:main May 16, 2024
@iamdmitrij
iamdmitrij deleted the IDE1006-fix branch May 16, 2024 11:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.