Skip to content

Remove redundant Attribute suffix - #24940

Merged
Ilya (iSazonov) merged 3 commits into
PowerShell:masterfrom
xtqqczze:rm-redundant-attribute-suffix
Feb 7, 2025
Merged

Remove redundant Attribute suffix#24940
Ilya (iSazonov) merged 3 commits into
PowerShell:masterfrom
xtqqczze:rm-redundant-attribute-suffix

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Feb 4, 2025

Copy link
Copy Markdown
Contributor

@iSazonov

This comment was marked as outdated.

@azure-pipelines

This comment was marked as outdated.

@iSazonov Ilya (iSazonov) added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Feb 6, 2025
@xtqqczze

xtqqczze commented Feb 7, 2025

Copy link
Copy Markdown
Contributor Author

3f99491: Fixed CodeFactor issue SA1411.

Related: #20087

@iSazonov

This comment was marked as outdated.

@azure-pipelines

This comment was marked as outdated.

@xtqqczze

xtqqczze commented Feb 7, 2025

Copy link
Copy Markdown
Contributor Author

PSResourceGet ACR failures are unrelated.

@iSazonov Ilya (iSazonov) self-assigned this Feb 7, 2025
@iSazonov
Ilya (iSazonov) merged commit 1790f3a into PowerShell:master Feb 7, 2025
@microsoft-github-policy-service

microsoft-github-policy-service Bot commented Feb 7, 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 rm-redundant-attribute-suffix branch February 7, 2025 17:32
@xtqqczze

xtqqczze commented Feb 7, 2025

Copy link
Copy Markdown
Contributor Author

Related PR to fix SA1411: #24969

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
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.

3 participants