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

Clean up testharness.css leftovers. #20718

Merged
merged 1 commit into from
Dec 11, 2019

Conversation

rakuco
Copy link
Member

@rakuco rakuco commented Dec 11, 2019

testharness.css itself was removed in #11143, but its .headers file
remained in the tree, and so did some lint checks for a file that no longer
exists.

testharness.css itself was removed in web-platform-tests#11143, but its `.headers` file
remained in the tree, and so did some lint checks for a file that no longer
exists.
Copy link
Member

@gsnedders gsnedders left a comment

Choose a reason for hiding this comment

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

I mean the lint is still valid, we don't want it there. 😛 But yeah, when there's nothing referring to it in tree already it seems unlikely anyone will add a new reference.

@gsnedders gsnedders merged commit 224eac4 into web-platform-tests:master Dec 11, 2019
@rakuco rakuco deleted the testrunnercss-cleanup branch December 11, 2019 16:44
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