Skip to content

Adding yml for Windows Release builds - #8374

Merged
Travis Plunk (TravisEz13) merged 23 commits into
PowerShell:masterfrom
anmenaga:WindowsBuild
Dec 7, 2018
Merged

Adding yml for Windows Release builds#8374
Travis Plunk (TravisEz13) merged 23 commits into
PowerShell:masterfrom
anmenaga:WindowsBuild

Conversation

@anmenaga

Copy link
Copy Markdown

PR Summary

As part of improving the build process, the UI build pipeline for Windows Builds was converted to a yml-based pipeline. (everything minus compliance tasks)
Verified on AzureDevOps mirror (pipeline WIP-PowerShellCore-WindowsBuild-Yaml): automated and manual runs.

PR Checklist

Comment thread tools/releaseBuild/azureDevOps/WindowsBuild.yml Outdated
Comment thread tools/releaseBuild/azureDevOps/WindowsBuild.yml
Comment thread tools/releaseBuild/azureDevOps/WindowsBuild.yml Outdated
Comment thread tools/releaseBuild/azureDevOps/WindowsBuild.yml
@anmenaga

Copy link
Copy Markdown
Author

Fixed CodeFactor issues.

@anmenaga

Copy link
Copy Markdown
Author

Aditya Patwardhan (@adityapatwardhan) pls look again.

@TravisEz13 Travis Plunk (TravisEz13) changed the title Adding yml for Windows builds Adding yml for Windows Release builds Dec 5, 2018
Comment thread tools/releaseBuild/azureDevOps/WindowsBuild.yml Outdated

@TravisEz13 Travis Plunk (TravisEz13) left a comment

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.

LGTM other than comments

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.

All comments are non-blocking


- powershell: |
docker container prune --force
docker container ls --all --format '{{ json .ID }}' | ConvertFrom-Json | %{docker container rm --force --volumes $_}

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.

Change % to ForEach-Object

condition: and(succeeded(), ne(variables['SkipFxDependent'], 'true'))

- powershell: |
docker image ls --format '{{ json .}}'|ConvertFrom-Json|% {

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.

Change % to ForEach-Object


- powershell: |
docker container prune --force
docker container ls --all --format '{{ json .ID }}' | ConvertFrom-Json | %{docker container rm --force --volumes $_}

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.

Change % to ForEach-Object

continueOnError: true

- powershell: |
docker image ls --format '{{ json .}}'|ConvertFrom-Json|% {

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.

Change % to ForEach-Object

@TravisEz13 Travis Plunk (TravisEz13) added the CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log label Dec 7, 2018
@TravisEz13
Travis Plunk (TravisEz13) merged commit a1299cd into PowerShell:master Dec 7, 2018
Travis Plunk (TravisEz13) pushed a commit to TravisEz13/PowerShell that referenced this pull request May 14, 2019
Thatgfsj (Thatgfsj) pushed a commit to Thatgfsj/PowerShell that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CL-BuildPackaging Indicates that a PR should be marked as a build or packaging change in the Change Log WG-Maintainers-Build specific to affecting the build

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants