fix #61 exclude option #62

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.

Sign up for GitHub

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

Jump to bottom

Open

cxy930123 wants to merge 1 commit into cuth:master from cxy930123:master

cxy930123 commented Jul 22, 2020

No description provided.


          fix #61 exclude option

959cfeb

zhang156 commented Aug 11, 2020

+1

wzdxy commented Dec 16, 2020

+1
I spent the whole afternoon debugging this bug.
I found this PR when I was about to modify the source code.

wzdxy commented Dec 16, 2020

When exclude is a function
filePath.match(exclude) always returns null , then all files be excluded

Owner

cuth commented Dec 16, 2020

Could you write a unit test to support this change?

defghy commented May 18, 2022

+1

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

Labels

None yet

pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.

Alternative Proxies:

Alternative Proxy