Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Addresses #3893 - Building PowerShell on Windows - Docs#3894

Closed
MarvTheRobot wants to merge 0 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
MarvTheRobot:masterMarvTheRobot/PowerShell:masterCopy head branch name to clipboard
Closed

Addresses #3893 - Building PowerShell on Windows - Docs#3894
MarvTheRobot wants to merge 0 commit into
PowerShell:masterPowerShell/PowerShell:masterfrom
MarvTheRobot:masterMarvTheRobot/PowerShell:masterCopy head branch name to clipboard

Conversation

@MarvTheRobot

Copy link
Copy Markdown
Contributor

windows-core.md is out of date with regards .Net Core CLI version being installed by Start-PSBootStrap.

Updated document with correct version number and channel where required.

@msftclas

Copy link
Copy Markdown

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@iSazonov

Copy link
Copy Markdown
Collaborator

@MarvTheRobot Thanks for the fix!

Note that we are ready to migrate to 006195 version with #3887 So I believe we should use this version in docs and hold the PR until #3887 is merged.

@MarvTheRobot

Copy link
Copy Markdown
Contributor Author

@iSazonov - agreed.

@mirichmo mirichmo self-assigned this May 31, 2017
@mirichmo

Copy link
Copy Markdown
Member

@MarvTheRobot #3887 is merged. Please update this PR so I can merge it..

@iSazonov

Copy link
Copy Markdown
Collaborator

@mirichmo

#3887 was reverted in #4026 😕

@MarvTheRobot

Copy link
Copy Markdown
Contributor Author

@iSazonov ; thanks, I was about to ask if it had worked as build.psm1 is still using 2.0.0-preview1-005952.

@MarvTheRobot

Copy link
Copy Markdown
Contributor Author

I'll re-raise this PR after I sort out my fork. I've been playing and broken it all

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.