-
Notifications
You must be signed in to change notification settings - Fork 384
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
Azure pipelines migration #1267
Merged
bergmeister
merged 33 commits into
PowerShell:master
from
bergmeister:AzurePipelinesMigration
Feb 2, 2020
Merged
Changes from all commits
Commits
Show all changes
33 commits
Select commit
Hold shift + click to select a range
1516ff5
make ci scripts agnostic of CI provider
bergmeister cc6464f
Merge branch 'master' of https://github.com/PowerShell/PSScriptAnalyz…
bergmeister a457ea7
Add basic yaml for testing in azure pipelines
bergmeister 16d1ae3
bootstrap template
bergmeister 8e3ded0
fix whitespace. the joy of yaml
bergmeister 91c4ca7
fix path
bergmeister 8025294
refactor
bergmeister ec67d89
test template
bergmeister e691f34
fix yaml
bergmeister a46dbd3
publish test results
bergmeister 3f150ba
fix indentation
bergmeister 15eacdb
unix yaml
bergmeister 7eda9ad
have bootstrap and build in the same task for unix
bergmeister d6bcc20
ps v5 template
bergmeister 79d3c85
ps v5 fix
bergmeister c55fd1e
try to use template for pwsh in bootstrap
bergmeister 6678f44
fix indent
bergmeister 3b4a63c
fix template param for psv2
bergmeister f0376d8
apply templating everywhere
bergmeister cb7037e
skip dontet cli init on v5 as well
bergmeister 7d4f0bf
Fix custom rules tests that fail on Linux
bergmeister 377be99
Add more assertion details to failure
bergmeister 4cff565
Merge branch 'master' of https://github.com/PowerShell/PSScriptAnalyz…
bergmeister 479a682
Add more assertion message to other failures and fix property name on…
bergmeister a42063e
Re-add calls to/PSCompatibilityCollector/build.ps1 -Configuration 'Re…
bergmeister 0289308
Merge branch 'master' into AzurePipelinesMigration
bergmeister 385febf
Exclude failing tests on AzDO/Ubuntu
ed81374
Merge pull request #11 from rjmholt/AzurePipelinesIntegration
bergmeister ccb4e91
try fix ubuntu build where dotnet --version makes the pipeline stop i…
bergmeister cc706d2
Merge branch 'master' of https://github.com/PowerShell/PSScriptAnalyz…
bergmeister a350326
Bootstrap as part of build process
bergmeister b44c9a9
remove bootstrap.yaml, which is now not needed any more
bergmeister bc53763
re-trigger sporadic failure
bergmeister File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,12 @@ | ||
variables: | ||
pwsh: true | ||
# Avoid expensive initialization of dotnet cli, see: https://donovanbrown.com/post/Stop-wasting-time-during-NET-Core-builds | ||
DOTNET_SKIP_FIRST_TIME_EXPERIENCE: 1 | ||
steps: | ||
- pwsh: | | ||
Import-Module .\tools\appveyor.psm1 | ||
Invoke-AppveyorInstall | ||
./build.ps1 -Configuration 'Release' -PSVersion 6 | ||
./PSCompatibilityCollector/build.ps1 -Configuration 'Release' -Framework 'netstandard2.0' | ||
displayName: 'Build' | ||
- template: templates/test.yaml |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
variables: | ||
pwsh: false | ||
# Avoid expensive initialization of dotnet cli, see: https://donovanbrown.com/post/Stop-wasting-time-during-NET-Core-builds | ||
DOTNET_SKIP_FIRST_TIME_EXPERIENCE: 1 | ||
steps: | ||
- powershell: | | ||
Import-Module .\tools\appveyor.psm1 | ||
Invoke-AppveyorInstall | ||
./build.ps1 -Configuration 'Release' -PSVersion 5 | ||
./PSCompatibilityCollector/build.ps1 -Configuration 'Release' -Framework 'net452' | ||
displayName: 'Build' | ||
- template: templates/test.yaml | ||
parameters: | ||
pwsh: $(pwsh) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
variables: | ||
pwsh: true | ||
# Avoid expensive initialization of dotnet cli, see: https://donovanbrown.com/post/Stop-wasting-time-during-NET-Core-builds | ||
DOTNET_SKIP_FIRST_TIME_EXPERIENCE: 1 | ||
steps: | ||
- pwsh: | | ||
Import-Module .\tools\appveyor.psm1 | ||
Invoke-AppveyorInstall | ||
./build.ps1 -Configuration 'Release' -All | ||
./PSCompatibilityCollector/build.ps1 -Configuration 'Release' -Framework 'netstandard2.0' | ||
displayName: 'Full Build' | ||
- template: templates/test.yaml | ||
parameters: | ||
pwsh: $(pwsh) |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
steps: | ||
- task: PowerShell@2 | ||
displayName: 'Test' | ||
inputs: | ||
targetType: inline | ||
pwsh: ${{ parameters.pwsh }} | ||
script: | | ||
Import-Module .\tools\appveyor.psm1 | ||
Invoke-AppveyorTest -CheckoutPath $env:BUILD_SOURCESDIRECTORY | ||
- task: PublishTestResults@2 | ||
inputs: | ||
testRunner: NUnit | ||
testResultsFiles: 'TestResults.xml' | ||
condition: succeededOrFailed() | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something that should be said is that although I set it to always run, but when the build or test fails, this task does not run and therefore test results are not uploaded. This seems to be a bug with the Azure DevOps YAML though. We should raise something but I don't think that's a blocking issue
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@JamesWTruher FYI