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

Adding win32 as an allowed OS #252

Merged
merged 1 commit into from Apr 27, 2012
Merged

Adding win32 as an allowed OS #252

merged 1 commit into from Apr 27, 2012

Conversation

@aaronpowell
Copy link
Contributor

aaronpowell commented Mar 21, 2012

I want to use this on windows under node and from testing it like this:

node ..\github\csslint\release\npm\cli.js .\style.css

It works just fine. I don't see why this can't be added as an allowed OS to the package.json file

@aaronpowell

This comment has been minimized.

Copy link
Contributor Author

aaronpowell commented Mar 21, 2012

I've done some research as I had heard it worked on Windows in the past. Turns out that since npm 1.1.8 they actually read the OS/ platform field - https://github.com/isaacs/npm/issues/1632 + https://github.com/isaacs/npm/issues/472

So now it does read it it'd be useful if it allowed Windows installs

@jzaefferer

This comment has been minimized.

Copy link

jzaefferer commented Apr 3, 2012

+1

@jzaefferer

This comment has been minimized.

Copy link

jzaefferer commented Apr 3, 2012

Or just drop the line.

@edwardbc

This comment has been minimized.

Copy link

edwardbc commented Apr 11, 2012

+1

1 similar comment
@doberkofler

This comment has been minimized.

Copy link

doberkofler commented Apr 15, 2012

+1

nzakas added a commit that referenced this pull request Apr 27, 2012
Adding win32 as an allowed OS
@nzakas nzakas merged commit 888b9d1 into CSSLint:master Apr 27, 2012
@edjafarov

This comment has been minimized.

Copy link

edjafarov commented on d6b0b97 May 9, 2012

could anyone republish the package in npm?

This comment has been minimized.

Copy link
Contributor

nzakas replied May 9, 2012

It will go out with the next release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
6 participants
You can’t perform that action at this time.