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

Added crawlStrict setting for finer detection in JS & CSS Files #30

Closed
wants to merge 2 commits into from

Conversation

madlandproject
Copy link

As explained in the updated README.md, This option makes crawling more precise ad avoids false positives if you are strict in the use of the Modernizr object in JS and HTML element scope in CSS.

… crawnling JS & CSS files.

JS tests are only detected when a property of the Modernizr object.
CSS/SASS/LESS tests are only detected when they are a class scoped to the HTML element
@rejas
Copy link
Member

rejas commented Mar 12, 2019

Hi @madlandproject sorry for not looking at PRs sooner :-( Are you still interested in pushing this PR? If so, could you rebase it against the latest master and make sure that the test-files are not included (those are noise). Maybe adding a new test for this feature would be nice though ;-)

@rejas
Copy link
Member

rejas commented Mar 16, 2019

Would fix #26

@rejas rejas closed this Feb 17, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants