Skip to content

Add configuration for Azure Pipelines#13769

Merged
foolip merged 14 commits intomasterfrom
foolip/azure-pipelines
Oct 31, 2018
Merged

Add configuration for Azure Pipelines#13769
foolip merged 14 commits intomasterfrom
foolip/azure-pipelines

Conversation

@foolip
Copy link
Member

@foolip foolip commented Oct 30, 2018

This just runs infrastructure/ tests on relevant PRs, something minimal
and useful before trying to enable Azure Pipelines for the first time.

Part of #13299.

This just runs infrastructure/ tests on relevant PRs, something minimal
and useful before trying to enable Azure Pipelines for the first time.

Part of #13299.
@foolip
Copy link
Member Author

foolip commented Oct 30, 2018

https://dev.azure.com/web-platform-tests/wpt is the Azure DevOps org/project where I'll add a pipeline to point at this file once landed. There is some minimal configuration that is done in that UI, like whether PRs trigger builds, but nothing too interesting.

@foolip
Copy link
Member Author

foolip commented Oct 30, 2018

https://github.com/foolip/wpt/pull/7/checks?check_run_id=27372593 / https://dev.azure.com/foolip/wpt/_build/results?buildId=118&view=logs is a run using this configuration, but with the condition lines deleted to make everything unconditional.

@foolip
Copy link
Member Author

foolip commented Oct 30, 2018

To make review easier if there are more changes, I've created foolip#10 from the very same branch, so that PR will be in sync with this one, but you can see Azure Pipelines results there.

@foolip
Copy link
Member Author

foolip commented Oct 31, 2018

This PR now builds on #13801, to be able to verify the config with a minimal change.

…re-pipelines"

This reverts commit 99b1f68, reversing
changes made to 93fedfc.
@foolip foolip mentioned this pull request Oct 31, 2018
@foolip foolip merged commit 4098fe5 into master Oct 31, 2018
@foolip foolip deleted the foolip/azure-pipelines branch October 31, 2018 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants