We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
I'm on windows 10 64bit -> process.platform == win32. But your code is:
https://github.com/jonschlinkert/is-invalid-path/blob/master/index.js#L11
const isWindows = (opts = {}) => process.platform !== 'win32' || opts.windows === true;
I had to change it to:
const isWindows = (opts = {}) => process.platform === 'win32' || opts.windows === true;
I'm confused to be the first to notice this :S
The text was updated successfully, but these errors were encountered:
Thanks, this definitely looks like a bug.
I think because this was introduced in a refactor and the major version was bumped, most of the people using this library are using the old version.
If you'd like to submit a PR with your change, that would be awesome! Otherwise, I'll try to get it updated soon.
Sorry, something went wrong.
Done! I didn't create it before because i was really confused it was set like that.
No branches or pull requests
I'm on windows 10 64bit -> process.platform == win32. But your code is:
https://github.com/jonschlinkert/is-invalid-path/blob/master/index.js#L11
I had to change it to:
I'm confused to be the first to notice this :S
The text was updated successfully, but these errors were encountered: