X Tutup
Skip to content

Code performance fixes#12956

Merged
iSazonov merged 4 commits intoPowerShell:masterfrom
xtqqczze:RCS-perf
Jun 25, 2020
Merged

Code performance fixes#12956
iSazonov merged 4 commits intoPowerShell:masterfrom
xtqqczze:RCS-perf

Conversation

@xtqqczze
Copy link
Copy Markdown
Contributor

@xtqqczze xtqqczze commented Jun 13, 2020

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 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 xtqqczze marked this pull request as ready for review June 19, 2020 18:30
@xtqqczze
Copy link
Copy Markdown
Contributor Author

@iSazonov rebased and fixed codefactor issues

@iSazonov 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 iSazonov added this to the 7.1.0-preview.4 milestone Jun 22, 2020
@iSazonov 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

ghost commented Jul 6, 2020

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

Handy links:

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

X Tutup