New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Replace GetDirectories in CimDscParser
#14319
Conversation
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
GetDirectories/GetFiles in CimDscParser
GetDirectories/GetFiles in CimDscParserGetDirectories in CimDscParser
|
CodeFactor issues are false positives. |
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
This pull request has been automatically marked as stale because it has been marked as requiring author feedback but has not had any activity for 15 days. It will be closed if no further activity occurs within 10 days of this comment. |
35ed26f
to
dd038a9
Compare
|
rebased to resolve CodeFactor issues |
|
2 years - time to merge |
|
|
|
This pull request has been automatically marked as Review Needed because it has been there has not been any activity for 7 days. |
|
@SteveL-MSFT Can you please re-review. |
|
/rebase |
Avoid array allocations from GetDirectories/GetFiles. Contributes to PowerShell#14318.
dd038a9
to
cba59fa
Compare
|
This PR has Quantification details
Why proper sizing of changes matters
Optimal pull request sizes drive a better predictable PR flow as they strike a
What can I do to optimize my changes
How to interpret the change counts in git diff output
Was this comment helpful? |
|
Handy links: |


Avoid array allocations from GetDirectories/GetFiles.
Contributes to #14318.