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

Minor refactoring of the release build scripts#5632

Merged
adityapatwardhan merged 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
daxian-dbw:builddaxian-dbw/PowerShell:buildCopy head branch name to clipboard
Dec 6, 2017
Merged

Minor refactoring of the release build scripts#5632
adityapatwardhan merged 3 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
daxian-dbw:builddaxian-dbw/PowerShell:buildCopy head branch name to clipboard

Conversation

@daxian-dbw

@daxian-dbw daxian-dbw commented Dec 5, 2017

Copy link
Copy Markdown
Member

PR Checklist

Note: Please mark anything not applicable to this PR NA.

PR Summary

Some name changes, comment updates, and random fixes.
This depends on PowerShell/PSRelease#28 to be merged.

Comment thread tools/releaseBuild/build.json Outdated
@@ -69,7 +69,7 @@
],
"DockerImageName": "ps-winsrvcore",
"BinaryBucket": "symbols",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

if we change the containerfolder in the other PR, we need to change the binarybucker where it equals symbols to results

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yes, the other PR has been updated. I also changed BinaryBucket to results for the -symbols entries.

@adityapatwardhan

Copy link
Copy Markdown
Member

@TravisEz13 Can you have another look?

@adityapatwardhan adityapatwardhan merged commit 233f35f into PowerShell:master Dec 6, 2017
TravisEz13 pushed a commit to TravisEz13/PowerShell that referenced this pull request Dec 7, 2017
* Update release build

* Minor refactoring of the build script
@daxian-dbw daxian-dbw deleted the build branch December 7, 2017 02:12
TravisEz13 pushed a commit that referenced this pull request Dec 7, 2017
* Update release build

* Minor refactoring of the build script
@TravisEz13 TravisEz13 added this to the 6.0.0-RC.2 milestone Dec 8, 2017
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.

3 participants

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