Skip to content
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

Expand numberOfPowershellRefAssemblies list capacity #12840

Merged
4 commits merged into from
Jun 2, 2020

Conversation

xtqqczze
Copy link
Contributor

@xtqqczze xtqqczze commented May 29, 2020

PR Summary

PR Context

HEAD of master has been broken since 99da109 (#12772), when .NET was updated to 5.0.100-preview.5.20278.13

#12815 (comment)

PR Checklist

@adityapatwardhan
Copy link
Member

@xtqqczze Please use a more descriptive PR title.

@xtqqczze xtqqczze changed the title Fix GH12815 WIP: Fix GH12815 May 29, 2020
@xtqqczze xtqqczze changed the title WIP: Fix GH12815 Expand numberOfPowershellRefAssemblies list capacity May 29, 2020
@xtqqczze
Copy link
Contributor Author

This resolves the issue in #12815 (comment) but does not fix #12815 as it does not include a static test.

@xtqqczze
Copy link
Contributor Author

@TravisEz13 can you help merge to resolve the breakage described in #12815 (comment)

@iSazonov iSazonov added the CL-CodeCleanup Indicates that a PR should be marked as a Code Cleanup change in the Change Log label May 31, 2020
@vexx32
Copy link
Collaborator

vexx32 commented Jun 1, 2020

@TravisEz13 and @adityapatwardhan probably best to get this merged sooner than later; folks can't currently run debug builds at all while this is an issue.

@xtqqczze
Copy link
Contributor Author

xtqqczze commented Jun 1, 2020

@iSazonov since you merged #12520, which this PR partially reverts, can you help merge this PR?

@TravisEz13 TravisEz13 added the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Jun 1, 2020
@ghost ghost removed the Waiting on Author The PR was reviewed and requires changes or comments from the author before being accept label Jun 1, 2020
@xtqqczze
Copy link
Contributor Author

xtqqczze commented Jun 1, 2020

The capacity is now 161 as per @iSazonov suggestion.

@xtqqczze
Copy link
Contributor Author

xtqqczze commented Jun 1, 2020

@TravisEz13 ready to merge?

@iSazonov iSazonov added the AutoMerge informs the bot to automerge the PR label Jun 2, 2020
@ghost
Copy link

ghost commented Jun 2, 2020

Hello @iSazonov!

Because this pull request has the AutoMerge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 8f7d308 into PowerShell:master Jun 2, 2020
@iSazonov iSazonov added this to the 7.1.0-preview.4 milestone Jun 2, 2020
@xtqqczze xtqqczze deleted the Fix-GH12815 branch June 2, 2020 11:31
@ghost
Copy link

ghost commented Jun 25, 2020

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

Handy links:

This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge informs the bot to automerge the PR 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.

None yet

5 participants