Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

增加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
kevinyan wants to merge 2 commits into runjuu:master from kevinyan:master
Closed

增加exclude #17

kevinyan wants to merge 2 commits into runjuu:master from kevinyan:master

Conversation

Copy link

@kevinyan kevinyan commented Oct 25, 2019

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

runjuu reacted with thumbs up emoji
如果不需要编译,需要用户自己写行数判断
Copy link
Owner

runjuu commented Nov 12, 2019
edited
Loading

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

whatever the name is
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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /