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

Update package.json #65

Merged
merged 1 commit into from
Dec 13, 2016
Merged

Update package.json #65

merged 1 commit into from
Dec 13, 2016

Conversation

antcook
Copy link

@antcook antcook commented Dec 13, 2016

On the auth-routes example the start script sets the node environment with NODE_ENV, which is not a recognised command on Windows. The work around is to use the cross-env package or to have two separate scripts, one for Windows and one for Unix based systems. I figured using the cross-env package is much more friendly and will save confusion, tested on both Windows and Mac.

NODE_ENV is not a recognised command on Windows, the work around is to use the cross-env package or to have two separate scripts, one for Windows and one for Unix based systems. I figured using the cross-env is much more friendly and will save confusion, tested on both Windows and Mac.
@codecov-io
Copy link

codecov-io commented Dec 13, 2016

Current coverage is 71.09% (diff: 100%)

Merging #65 into master will not change coverage

@@             master        #65   diff @@
==========================================
  Files            10         10          
  Lines           391        391          
  Methods           0          0          
  Messages          0          0          
  Branches          0          0          
==========================================
  Hits            278        278          
  Misses          113        113          
  Partials          0          0          

Powered by Codecov. Last update 0f81a13...71b6b99

@Atinux Atinux merged commit ec581ce into nuxt:master Dec 13, 2016
@Atinux
Copy link
Member

Atinux commented Dec 13, 2016

Thank you @antcook

Will be added to the next release 👍

@lock
Copy link

lock bot commented Nov 1, 2018

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock bot locked as resolved and limited conversation to collaborators Nov 1, 2018
@danielroe danielroe added the 2.x label Jan 18, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants