search

twbs / bootlint

HTML linter for Bootstrap projects
MIT License
2.4k stars 312 forks source link

Different html input on Travis and on Windows #406

Closed XhmikosR closed 7 years ago

XhmikosR commented 7 years ago

So, in Travis we do:

node ./src/cli-main.js bootstrap/_gh_pages/**/index.html

which results in:

$ node ./src/cli-main.js bootstrap/_gh_pages/**/index.html

0 lint error(s) found across 8 file(s).

If I run the same on Windows 7, I get:

C:\Users\xmr\Desktop\bootlint>node ./src/cli-main.js bootstrap/_gh_pages/**/index.html
bootstrap/_gh_pages/examples/grid/index.html: W005 Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work; however, you might not be using Bootstrap's JavaScript
bootstrap/_gh_pages/examples/jumbotron-narrow/index.html: W005 Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work; however, you might not be using Bootstrap's JavaScript
bootstrap/_gh_pages/examples/justified-nav/index.html: W005 Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work; however, you might not be using Bootstrap's JavaScript
bootstrap/_gh_pages/examples/non-responsive/index.html: W003 `<head>` is missing viewport `<meta>` tag that enables responsiveness
bootstrap/_gh_pages/examples/signin/index.html: W005 Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work; however, you might not be using Bootstrap's JavaScript
bootstrap/_gh_pages/examples/sticky-footer/index.html: W005 Unable to locate jQuery, which is required for Bootstrap's JavaScript plugins to work; however, you might not be using Bootstrap's JavaScript

For details, look up the lint problem IDs in the Bootlint wiki: https://github.com/twbs/bootlint/wiki
6 lint error(s) found across 28 file(s).
XhmikosR commented 7 years ago

OK found it. It's because the glob is unquoted.

I'll merge my branch which fixes this.