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

Add // META: spec=link or remove MISSING-LINK css lint #5770

Open
zcorpan opened this Issue May 3, 2017 · 3 comments

Comments

Projects
None yet
5 participants
@zcorpan
Copy link
Contributor

zcorpan commented May 3, 2017

See #5769

It should be possible to use .worker.js or .any.js under css/ without having to whitelist for MISSING-LINK lint failure. Either we add a new // META: thing to specify a link, or we silently allow such tests without a link.

@gsnedders

@gsnedders

This comment has been minimized.

Copy link
Contributor

gsnedders commented May 3, 2017

We have the MISSING-LINK lint to make sure the CSS build system picks up everything, and that doesn't support .worker.js or .any.js tests. Um. idk what the right solution is here? Well, apart from replace the CSS build system with something saner.

@foolip

This comment has been minimized.

Copy link
Contributor

foolip commented Jan 3, 2018

@gsnedders are you planning to work on this? (Came up in triage of roadmap issues older than 60 days.)

@lukebjerring

This comment has been minimized.

Copy link
Contributor

lukebjerring commented Apr 3, 2018

This is blocking #9884

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment