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

Removing foreign fetch tests #7762

Merged
merged 2 commits into from Nov 22, 2017

Conversation

Projects
None yet
4 participants
@jakearchibald
Contributor

jakearchibald commented Oct 13, 2017

This feature is being removed from the spec w3c/ServiceWorker#1207.

@w3c-bots

This comment has been minimized.

Show comment
Hide comment
@w3c-bots

w3c-bots Oct 13, 2017

Build PASSED

Started: 2017-11-16 18:08:48
Finished: 2017-11-16 18:16:31

View more information about this build on:

w3c-bots commented Oct 13, 2017

Build PASSED

Started: 2017-11-16 18:08:48
Finished: 2017-11-16 18:16:31

View more information about this build on:

@jdm

jdm approved these changes Oct 13, 2017

@jakearchibald

This comment has been minimized.

Show comment
Hide comment
@jakearchibald

jakearchibald Oct 25, 2017

Contributor

We're deciding whether we keep InstallEvent or not. If not, I should remove it from the tests as part of this PR.

Contributor

jakearchibald commented Oct 25, 2017

We're deciding whether we keep InstallEvent or not. If not, I should remove it from the tests as part of this PR.

@jakearchibald

This comment has been minimized.

Show comment
Hide comment
@jakearchibald

jakearchibald Nov 16, 2017

Contributor

Sorry about the noise. I botched a rebase.

Contributor

jakearchibald commented Nov 16, 2017

Sorry about the noise. I botched a rebase.

@wpt-pr-bot wpt-pr-bot requested review from ayg, ehsan, jgraham, mattto and zqzhang Nov 16, 2017

@jakearchibald jakearchibald merged commit 1433896 into web-platform-tests:master Nov 22, 2017

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details

annevk added a commit to whatwg/html that referenced this pull request Nov 22, 2017

Remove <link rel=serviceworker> for now
This is being removed as part of foreign fetch, but we may add it again once we have a better understanding of the use cases.

Tests removed in: web-platform-tests/wpt#7762.

More context: w3c/ServiceWorker#1207.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment