Skip to content

增加exclude #17

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

Closed
wants to merge 2 commits into from
Closed

增加exclude #17

wants to merge 2 commits into from

Conversation

kevinyan
Copy link

有些场景,除了不编译之外,还有不处理的需求。
使用过程中发现 filter把不要CSS直接过滤掉了,而常见需求是不需要inline。所以处理一下,看是否合理

如果不需要编译,需要用户自己写行数判断
@runjuu
Copy link
Owner

runjuu commented Nov 12, 2019

Hi @kevinyan ,现在确实存在这个问题,但感觉 exclude 的命名和 filter 同时存在有点让人迷惑。可能我们需要想另外一个名称,感觉叫 ignore 会更直观一点?但或许单纯的动词没法准确地描述它们的行为。

whatever the name is
@github-actions
Copy link

Hello 👋, this PR has been opened for more than 2 months with no activity on it. If you think this is a mistake please comment and ping a maintainer to get this merged ASAP! Thanks for contributing! You have 7 days until this gets closed automatically

@github-actions github-actions bot added the Stale label Jan 19, 2020
@github-actions github-actions bot closed this Jan 27, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants