Skip to content
This repository has been archived by the owner on Aug 19, 2022. It is now read-only.

Fix postinstall on windows #186

Merged
merged 1 commit into from Jun 1, 2015
Merged

Fix postinstall on windows #186

merged 1 commit into from Jun 1, 2015

Conversation

ianobermiller
Copy link
Contributor

Creating a PR to test the appveyor build

@ryan-roemer
Copy link
Member

@ianobermiller -- If appveyor doesn't like this another option is to just use npm programmatically in a Node script. Here's some totally unrelated code that looks for extra devDependencies that for Heroku only should be installed and then uses NPM programmatically to install them... https://github.com/FormidableLabs/full-stack-testing/blob/master/app/heroku/install.js

(Just in case it helps ;) )

@ianobermiller
Copy link
Contributor Author

Probably running into this: nodejs/node-v0.x-archive#2318

@ianobermiller ianobermiller changed the title Try npm.bat on windows Fix npm spawn in package.json on windows May 31, 2015
@ianobermiller ianobermiller changed the title Fix npm spawn in package.json on windows Fix postinstall on windows May 31, 2015
@ianobermiller
Copy link
Contributor Author

Yes!

ianobermiller added a commit that referenced this pull request Jun 1, 2015
@ianobermiller ianobermiller merged commit 4daf6d7 into master Jun 1, 2015
@ianobermiller ianobermiller deleted the windows-build branch June 1, 2015 13:50
@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 5995703 on windows-build into 8a7a821 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling f0dc4a0 on windows-build into 8a7a821 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 23ab310 on windows-build into 8a7a821 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 06fc986 on windows-build into 5634b9f on master.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants