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

Disable IDE0049: PreferBuiltInOrFrameworkType#26094

Merged
iSazonov merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
xtqqczze:disable-IDE0049xtqqczze/PowerShell-PowerShell:disable-IDE0049Copy head branch name to clipboard
Oct 10, 2025
Merged

Disable IDE0049: PreferBuiltInOrFrameworkType#26094
iSazonov merged 1 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
xtqqczze:disable-IDE0049xtqqczze/PowerShell-PowerShell:disable-IDE0049Copy head branch name to clipboard

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Sep 25, 2025

Copy link
Copy Markdown
Contributor

Temporarily disable this rule until all violations are addressed.

Not all rule violations were resolved in #14491. Since the rule has internal setting EnforceOnBuild.Never it doesn’t affect builds, however it still produces distracting warnings during design-time analysis.

https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0049

Related: #25922

@iSazonov

iSazonov commented Oct 9, 2025

Copy link
Copy Markdown
Collaborator

@xtqqczze Please rebase.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Review - Needed The PR is being reviewed label Oct 9, 2025
Temporarily disable this rule until all violations are addressed.

Not all rule violations were resolved in PowerShell#14491. Since the rulehas internal setting `EnforceOnBuild.Never` it doesn’t affect builds, however it still produces distracting warnings during design-time analysis.

https://docs.microsoft.com/dotnet/fundamentals/code-analysis/style-rules/ide0049
@iSazonov iSazonov added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Oct 10, 2025
@iSazonov iSazonov self-assigned this Oct 10, 2025
@iSazonov

Copy link
Copy Markdown
Collaborator

/azp run PowerShell-Windows-Packaging-CI, PowerShell-CI-linux-packaging

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines could not run because the pipeline triggers exclude this branch/path.

@iSazonov iSazonov enabled auto-merge (squash) October 10, 2025 04:17
@iSazonov iSazonov merged commit 3207a74 into PowerShell:master Oct 10, 2025
36 of 37 checks passed
@microsoft-github-policy-service

microsoft-github-policy-service Bot commented Oct 10, 2025

Copy link
Copy Markdown
Contributor

📣 Hey @@xtqqczze, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗 https://aka.ms/PSRepoFeedback

@xtqqczze xtqqczze deleted the disable-IDE0049 branch October 10, 2025 09:59
SIRMARGIN pushed a commit to SIRMARGIN/PowerShell that referenced this pull request Dec 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log

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.