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

[FileAPI] Remove reference to non-existent file #14132

Merged
merged 2 commits into from
Nov 26, 2018

Conversation

jugglinmike
Copy link
Contributor

The file FileAPI/url/url-with-tags.window.js is not currently present
in the repository nor was it present when the reference was introduced
[1]. Remove it to avoid triggering spurious request failures during test
execution.

[1] 9d8a185

The file `FileAPI/url/url-with-tags.window.js` is not currently present
in the repository nor was it present when the reference was introduced
[1]. Remove it to avoid triggering spurious request failures during test
execution.

[1] 9d8a185
Copy link
Contributor

@mkruisselbrink mkruisselbrink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ah, good catch. I think I meant that to be url-in-tags.window.js

@jugglinmike
Copy link
Contributor Author

Got it. I've added a reference to that file via another commit. The commit message will have to be updated, but before that, we have an instability on our hands.

Bisecting indicates that it's a wptrunner regression from 31664a8, which was merged via gh-13419. A newer wptrunner patch is currently under review, and as luck would have it, it fixes this problem.

This change should pass the stability checks once that patch is merged.

@Ms2ger Ms2ger merged commit 73167bb into web-platform-tests:master Nov 26, 2018
@jugglinmike
Copy link
Contributor Author

@Ms2ger This branch has a commit prefixed with fixup!; mind squashing those prior to landing in the future? I'd also be happy to do that and force-push to the branch if that's your preference.

@Ms2ger
Copy link
Contributor

Ms2ger commented Nov 27, 2018

Sure; I overlooked the commit here.

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.

4 participants