Skip to content

Replace ArrayList with List<T>#10333

Merged
adityapatwardhan merged 9 commits intoPowerShell:masterfrom
iSazonov:cleanup-replace-arraylist
Sep 5, 2019
Merged

Replace ArrayList with List<T>#10333
adityapatwardhan merged 9 commits intoPowerShell:masterfrom
iSazonov:cleanup-replace-arraylist

Conversation

@iSazonov
Copy link
Copy Markdown
Collaborator

@iSazonov iSazonov commented Aug 9, 2019

PR Summary

Related #10315

Replace ArrayList where we use it unjustifiably to get more readable and reliable code.

PR Context

PR Checklist

@iSazonov iSazonov added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label Aug 9, 2019
@iSazonov iSazonov added this to the 7.0.0-preview.3 milestone Aug 9, 2019
@iSazonov iSazonov force-pushed the cleanup-replace-arraylist branch 2 times, most recently from 0ac1d7b to a2e4fd7 Compare August 9, 2019 12:24
@iSazonov
Copy link
Copy Markdown
Collaborator Author

@daxian-dbw @anmenaga Could you please review?

@iSazonov iSazonov force-pushed the cleanup-replace-arraylist branch from 8418d2d to 0e9857a Compare August 30, 2019 07:41
@iSazonov
Copy link
Copy Markdown
Collaborator Author

iSazonov commented Sep 3, 2019

@adityapatwardhan Could you please review the small style PR?

@adityapatwardhan adityapatwardhan merged commit fbf4f6c into PowerShell:master Sep 5, 2019
@iSazonov iSazonov deleted the cleanup-replace-arraylist branch September 6, 2019 03:42
TravisEz13 pushed a commit to TravisEz13/PowerShell that referenced this pull request Sep 14, 2019
@ghost
Copy link
Copy Markdown

ghost commented Sep 19, 2019

🎉v7.0.0-preview.4 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.

5 participants