Skip to content

Remove redundant Attribute suffix#24940

Merged
iSazonov merged 3 commits intoPowerShell:masterfrom
xtqqczze:rm-redundant-attribute-suffix
Feb 7, 2025
Merged

Remove redundant Attribute suffix#24940
iSazonov merged 3 commits intoPowerShell:masterfrom
xtqqczze:rm-redundant-attribute-suffix

Conversation

@xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented Feb 4, 2025

@iSazonov

This comment was marked as outdated.

@azure-pipelines

This comment was marked as outdated.

@iSazonov 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
Copy link
Contributor Author

xtqqczze commented Feb 7, 2025

3f99491: Fixed CodeFactor issue SA1411.

Related: #20087

@iSazonov

This comment was marked as outdated.

@azure-pipelines

This comment was marked as outdated.

@xtqqczze
Copy link
Contributor Author

xtqqczze commented Feb 7, 2025

PSResourceGet ACR failures are unrelated.

@iSazonov iSazonov self-assigned this Feb 7, 2025
@iSazonov iSazonov merged commit 1790f3a into PowerShell:master Feb 7, 2025
39 of 41 checks passed
@microsoft-github-policy-service
Copy link
Contributor

microsoft-github-policy-service bot commented Feb 7, 2025

📣 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
Copy link
Contributor Author

xtqqczze commented Feb 7, 2025

Related PR to fix SA1411: #24969

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