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

Add configuration for Azure Pipelines #13769

Merged
merged 14 commits into from
Oct 31, 2018
Merged

Add configuration for Azure Pipelines #13769

merged 14 commits into from
Oct 31, 2018

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.

@foolip
Add configuration for Azure Pipelines
1a75ef5 
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.
@wpt-pr-bot wpt-pr-bot requested a review from jgraham October 30, 2018 00:02
@foolip foolip requested a review from thejohnjansen October 30, 2018 00:03
@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 foolip mentioned this pull request Oct 30, 2018
Closed
@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.

jgraham
jgraham reviewed Oct 30, 2018
View reviewed changes
.azure-pipelines.yml Show resolved Hide resolved
.azure-pipelines.yml Outdated Show resolved Hide resolved
@foolip foolip mentioned this pull request Oct 30, 2018
Closed
@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.

@wpt-pr-bot wpt-pr-bot added the ci label Oct 30, 2018
@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.

@foolip foolip mentioned this pull request Oct 31, 2018
Open
@foolip
Revert "REVERTME: Merge branch 'foolip/test-jobs-all' into foolip/azu…
23f442b 
…re-pipelines"

This reverts commit 99b1f68, reversing
changes made to 93fedfc.
@foolip foolip mentioned this pull request Oct 31, 2018
Closed
@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
Reviewers

@jgraham jgraham jgraham approved these changes

@thejohnjansen thejohnjansen Awaiting requested review from thejohnjansen

@gsnedders gsnedders Awaiting requested review from gsnedders

@jugglinmike jugglinmike Awaiting requested review from jugglinmike

Assignees
No one assigned
Labels
Azure Pipelines ci infra
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@foolip @jgraham @wpt-pr-bot