Skip to content

Code performance fixes - #12956

Merged
Ilya (iSazonov) merged 4 commits into
PowerShell:masterfrom
xtqqczze:RCS-perf
Jun 25, 2020
Merged

Code performance fixes#12956
Ilya (iSazonov) merged 4 commits into
PowerShell:masterfrom
xtqqczze:RCS-perf

Conversation

@xtqqczze

@xtqqczze xtqqczze commented Jun 13, 2020

Copy link
Copy Markdown
Contributor

PR Summary

  • Autofix RCS1080: Use 'Count/Length' property instead of 'Any' method
  • Autofix RCS1077: Optimize LINQ method call
  • Autofix RCS1235: Optimize method call

PR Context

PR Checklist

@ghost ghost assigned Ilya (iSazonov) Jun 13, 2020
@xtqqczze xtqqczze changed the title Rcs perf Code performance fixes Jun 15, 2020
@iSazonov

Copy link
Copy Markdown
Collaborator

xtqqczze Please rebase to pass CIs.

@iSazonov

Copy link
Copy Markdown
Collaborator

xtqqczze Please rebase.

@xtqqczze

Copy link
Copy Markdown
Contributor Author

Ilya (@iSazonov) rebased and fixed codefactor issues

@iSazonov Ilya (iSazonov) added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Jun 22, 2020
@iSazonov Ilya (iSazonov) added this to the 7.1.0-preview.4 milestone Jun 22, 2020
@iSazonov
Ilya (iSazonov) merged commit bd90f19 into PowerShell:master Jun 25, 2020
@xtqqczze
xtqqczze deleted the RCS-perf branch June 29, 2020 13:08
@ghost

Copy link
Copy Markdown

🎉v7.1.0-preview.5 has been released which incorporates this pull request.:tada:

Handy links:

Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
* RCS1080: Use 'Count/Length' property instead of 'Any' method

* RCS1077: Optimize LINQ method call

* RCS1235: Optimize method call
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