8000 Make the variable an `ArrayList` to enable usage of `Add()` method by adityapatwardhan · Pull Request #14748 · PowerShell/PowerShell · GitHub
[go: up one dir, main page]
Skip to content

Make the variable an ArrayList to enable usage of Add() method #14748

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.

Sign up for GitHub

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

Jump to bottom
Merged
adityapatwardhan merged 1 commit into from
Feb 9, 2021
Merged

Make the variable an ArrayList to enable usage of Add() method #14748

adityapatwardhan merged 1 commit into from
Feb 9, 2021

Conversation

@adityapatwardhan
Copy link
Member
@adityapatwardhan adityapatwardhan commented Feb 9, 2021 •
edited
Loading

PR Summary

Fix the build issues while adding experimental features to the list of features.

PR Context

Regression introduced in #13399

PR Checklist

@ghost ghost assigned rjmholt Feb 9, 2021
@adityapatwardhan adityapatwardhan added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Feb 9, 2021
@rjmholt
Copy link
Collaborator
rjmholt commented Feb 9, 2021

Waiting on #14750

@adityapatwardhan
Copy link
Member Author
adityapatwardhan commented Feb 9, 2021

@rjmholt #14750 is merged and CI is re-running

@adityapatwardhan adityapatwardhan merged commit 894db0f into PowerShell:master Feb 9, 2021
@adityapatwardhan adityapatwardhan deleted the FixExpBuildIssue branch February 9, 2021 21:59
@adityapatwardhan adityapatwardhan added this to the 7.2.0-preview.3 milestone Feb 9, 2021
@ghost
Copy link
ghost commented Feb 12, 2021

🎉v7.2.0-preview.3 has been released which incorporates this pull request.:tada:

Handy links:

Steg17
Steg17 reviewed Feb 13, 2021
View reviewed changes
Copy link
@Steg17 Steg17 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@anmenaga anmenaga mentioned this pull request Feb 19, 2021
Merged
21 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@daxian-dbw daxian-dbw daxian-dbw approved these changes

@TravisEz13 TravisEz13 Awaiting requested review from TravisEz13

+2 more reviewers

@Steg17 Steg17 Steg17 left review comments

@anmenaga anmenaga anmenaga approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

@rjmholt rjmholt

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log

Projects

None yet

Milestone

7.2.0-preview.4

Development

Successfully merging this pull request may close these issues.

6 participants

@adityapatwardhan @rjmholt @daxian-dbw @anmenaga @Steg17 @TravisEz13
0