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

Rename main script to "redux" to make it work in cmd. #67

Merged
merged 1 commit into from
May 3, 2016
Merged

Rename main script to "redux" to make it work in cmd. #67

merged 1 commit into from
May 3, 2016

Conversation

tornewuff
Copy link
Contributor

The batch scripts generated by "npm install" on Windows run the module
using its actual filename, not the name of the installed binary. This
means that argv[1] ends up being set to "index" and commander can't run
any subcommands. Rename the main module to "redux" so that this matches.

Fixes #54.

The batch scripts generated by "npm install" on Windows run the module
using its actual filename, not the name of the installed binary. This
means that argv[1] ends up being set to "index" and commander can't run
any subcommands. Rename the main module to "redux" so that this matches.
@codecov-io
Copy link

Current coverage is 71.27%

Merging #67 into master will decrease coverage by -5.18%

  1. 3 files (not in diff) in src/util were modified. more
    • Hits -10
  2. 6 files (not in diff) in src/models were modified. more
    • Misses -2
    • Hits -57
@@             master        #67   diff @@
==========================================
  Files            11         11          
  Lines           344        275    -69   
  Methods          28         28          
  Messages          0          0          
  Branches         45         45          
==========================================
- Hits            263        196    -67   
+ Misses           81         79     -2   
  Partials          0          0          

Powered by Codecov. Last updated by c8b11ab...76141a2

@SpencerCDixon
Copy link
Owner

Awesome, thank you so much for this! Going to merge tonight when I get home on my personal computer

@SpencerCDixon SpencerCDixon merged commit 7770a44 into SpencerCDixon:master May 3, 2016
fp-x added a commit to fp-x/youbase that referenced this pull request Apr 15, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants