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.

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

Multi-stage pipelines (to split build->test) #1404

Merged
merged 36 commits into from
Feb 4, 2020
Merged

Multi-stage pipelines (to split build->test) #1404

merged 36 commits into from
Feb 4, 2020

Conversation

bergmeister
Copy link
Collaborator

@bergmeister bergmeister commented Feb 3, 2020 •
edited
Loading

PR Summary

This also makes the test matrix much bigger because it's free due to parallelism.

Somehow this also exposed a small bug in a rule around newlines that I had to fix as part of it.

PR Checklist

@bergmeister bergmeister changed the title WIP: Multi stage yaml WIP: Multi-stage pipelines (to split build->test) Feb 3, 2020
@bergmeister bergmeister changed the title WIP: Multi-stage pipelines (to split build->test) Multi-stage pipelines (to split build->test) Feb 3, 2020
@bergmeister bergmeister changed the title Multi-stage pipelines (to split build->test) WIP Multi-stage pipelines (to split build->test) Feb 3, 2020
@bergmeister bergmeister changed the title WIP Multi-stage pipelines (to split build->test) Multi-stage pipelines (to split build->test) Feb 3, 2020
@bergmeister bergmeister marked this pull request as ready for review February 3, 2020 23:16
Merge branch 'master' of https://github.com/powershell/psscriptanalyzer…
20e5b1b 
… into multi-stage-yaml

# Conflicts:
#	.azure-pipelines-ci/ci.yaml
#	.azure-pipelines-ci/templates/unix.yaml
rjmholt
rjmholt approved these changes Feb 4, 2020
View reviewed changes
.azure-pipelines-ci/ci.yaml Outdated Show resolved Hide resolved
@bergmeister bergmeister merged commit 1305c88 into PowerShell:master Feb 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rjmholt rjmholt rjmholt approved these changes

@JamesWTruher JamesWTruher Awaiting requested review from JamesWTruher

Assignees
No one assigned
Labels
Area - Build
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@bergmeister @rjmholt