You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
On Windows yamllint does not work since v1.3.0. Issue is that globbySync does not understand patterns with back slashes, see related issue sindresorhus/globby#130.
Even if you run yamllint with forward slashes e.g. yamllint some/path/to/my.yaml, then call of resolve(process.cwd(), pattern) breaks it because converts path to Windows style i.e. some\path\to\my.yaml.
Simple solution is to use something like resolve(process.cwd(), pattern).replace(/\\/g, '/').
The text was updated successfully, but these errors were encountered:
Oops, sorry about that. daa4899 fixes the issue and I extended the CI pipeline to also run the tests in Windows and macOS to catch such issues in the future. The fixed version has been released as v1.7.0 a few seconds ago.
Thanks for the detailed report, much appreciated! 👍
On Windows yamllint does not work since v1.3.0. Issue is that globbySync does not understand patterns with back slashes, see related issue sindresorhus/globby#130.
Even if you run yamllint with forward slashes e.g.
yamllint some/path/to/my.yaml
, then call ofresolve(process.cwd(), pattern)
breaks it because converts path to Windows style i.e.some\path\to\my.yaml
.Simple solution is to use something like
resolve(process.cwd(), pattern).replace(/\\/g, '/')
.The text was updated successfully, but these errors were encountered: