Remove unnecessary `CS0618` suppressions from Variant APIs by xtqqczze · Pull Request #26006 · PowerShell/PowerShell

Skip to content

Navigation Menu

Sign in

Appearance settings

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

Sign up

Appearance settings

Conversation

@xtqqczze

Copy link

Contributor

@xtqqczze xtqqczze commented

Sep 7, 2025

edited

Loading

The VarEnum and COM-marshaling APIs have had the obsolete attribute removed, therefore CS0618 suppression is unnecessary.

References: dotnet/corefx#35161

@iSazonov

Copy link

Collaborator

iSazonov commented

Sep 7, 2025

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

@azure-pipelines

Copy link

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

@iSazonov iSazonov added the CL-CodeCleanup

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

label

Sep 7, 2025

@iSazonov iSazonov enabled auto-merge (squash)

September 7, 2025 14:21

@iSazonov iSazonov merged commit 43c2d98 into PowerShell:master

Sep 7, 2025

35 of 39 checks passed

@microsoft-github-policy-service

Copy link

Contributor

microsoft-github-policy-service bot commented

Sep 7, 2025

edited by unfurl-links bot

Loading

📣 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 cs0618-variant branch

September 7, 2025 14:55

@iSazonov iSazonov self-assigned this

Oct 9, 2025

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

Reviewers

@iSazonov iSazonov iSazonov approved these changes

Assignees

@iSazonov iSazonov

Labels

CL-CodeCleanup

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

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xtqqczze @iSazonov