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.

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

[GitHub Actions] trigger push workflows on push to master only #18924

Merged
merged 1 commit into from Sep 10, 2019

Conversation

@foolip
Copy link
Member

@foolip foolip commented Sep 9, 2019

These are being triggered on pushes to all branches now, including
epochs/* and triggers/* branches. That's not useful.

These are being triggered on pushes to all branches now, including
epochs/* and triggers/* branches. That's not useful.
@wpt-pr-bot
Copy link
Collaborator

@wpt-pr-bot wpt-pr-bot commented Sep 9, 2019

There are no reviewers for this pull request. Please reach out on W3C's irc server (irc.w3.org, port 6665) on channel #testing (web client) to get help with this. Thank you!

@foolip
Copy link
Member Author

@foolip foolip commented Sep 9, 2019

@jgraham can you review? Perhaps it's an intended side effect that some of the steps get run on PRs for branches in this repo, and that most changes that could break this are from such PRs? If so we should run these on PRs too. I'd like to think that's not necessary for at least the manifest workflow, but... opinions?

@foolip foolip merged commit 8d867f8 into master Sep 10, 2019
14 checks passed
14 checks passed
update-pr-preview
Details
Azure Pipelines Build #20190909.31 succeeded
Details
Azure Pipelines (./wpt test-jobs) ./wpt test-jobs succeeded
Details
Azure Pipelines (affected tests without changes: Safari Technology Preview) affected tests without changes: Safari Technology Preview succeeded
Details
Azure Pipelines (affected tests: Safari Technology Preview) affected tests: Safari Technology Preview succeeded
Details
Azure Pipelines (wpt.fyi hook: safari-preview-affected-tests) wpt.fyi hook: safari-preview-affected-tests succeeded
Details
Azure Pipelines (wpt.fyi hook: safari-preview-affected-tests-without-changes) wpt.fyi hook: safari-preview-affected-tests-without-changes succeeded
Details
Taskcluster (pull_request) TaskGroup: success
Details
staging.wpt.fyi - chrome[experimental] Chrome results
Details
staging.wpt.fyi - firefox[experimental] Firefox results
Details
staging.wpt.fyi - safari[experimental] Safari results
Details
wpt.fyi - chrome[experimental] Chrome results
Details
wpt.fyi - firefox[experimental] Firefox results
Details
wpt.fyi - safari[experimental] Safari results
Details
@foolip foolip deleted the foolip/github-actions-push-master branch Sep 10, 2019
@zcorpan zcorpan added GitHub Actions and removed .github labels Oct 25, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.