Skip to content

small fix #47

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

Merged
merged 2 commits into from
Aug 24, 2016
Merged

small fix #47

merged 2 commits into from
Aug 24, 2016

Conversation

lionel-bijaoui
Copy link
Member

Very small stuff, should be quickly merge if you agree with the changes

Lionel Bijaoui added 2 commits August 24, 2016 15:30
@coveralls
Copy link

coveralls commented Aug 24, 2016

Coverage Status

Coverage remained the same at 81.4% when pulling 04a71a0 on lionel-bijaoui:LB_small_fix into dcfde9f on icebob:master.

@@ -17,7 +17,7 @@
"e2e": "node test/e2e/runner.js",
"pretest": "npm run lint",
"test": "npm run unit",
"ci": "karma start test/unit/karma.conf.js --auto-watch --no-single-run --reporters=dots"
"ci": "karma start test/unit/karma.conf.js --auto-watch --no-single-run --reporters=dots --port=9877"
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why need it?

Copy link
Member Author

@lionel-bijaoui lionel-bijaoui Aug 24, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

As I said in the commit, It allow to use both npm run test and npm run ci at the same time without warning. It is a very small stuff, but since I use them all the time, I want to avoid any interference even if very unlikely.

@icebob icebob merged commit 750afd4 into vue-generators:master Aug 24, 2016
@icebob
Copy link
Member

icebob commented Aug 24, 2016

ok, merged

@lionel-bijaoui lionel-bijaoui deleted the LB_small_fix branch August 24, 2016 13:53
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.

3 participants