Skip to content
This repository has been archived by the owner on Jun 27, 2019. It is now read-only.

Remove nyc as a global dependency #82

Merged
merged 3 commits into from
Jun 30, 2018
Merged

Remove nyc as a global dependency #82

merged 3 commits into from
Jun 30, 2018

Conversation

roschaefer
Copy link
Contributor

Local installation over global installation, no? 🤔

@appinteractive I don't see the difference between test and test:coverage. On current develop branch the two results look like the same.

@appinteractive
Copy link
Member

The results yes but did you get the same results in the coverage folder?

The local installation of nyc does not work for me at all.

@roschaefer
Copy link
Contributor Author

OK, so test:coverage creates the file lcov.info in coverage/? That does not work locally on my machine. The tests behave as expected though.

I guess I have to research some more on how to chain scripts in package.json. I find the current package.json quite confusing and would like to clean it up a little.

@appinteractive
Copy link
Member

appinteractive commented Jun 29, 2018 via email

@appinteractive
Copy link
Member

I don't know why but intalling nyc localy always throws this:
grafik
Even in yarn dev

@ghost
Copy link

ghost commented Jun 29, 2018

@appinteractive just found this: npm/npmlog#48

seems nuking node_modules works in some cases. In others there are dependency conflicts where packages require different versions of gauge where older versions do not have isEnabled as a function.

@appinteractive
Copy link
Member

@sebako86 funny I just was there and saw your reference (two min before) ^^
Sadly that does not help in my case.

@appinteractive
Copy link
Member

appinteractive commented Jun 29, 2018 via email

@appinteractive
Copy link
Member

Works for everyone except me so I will merge it and still install it for me globally for now.

@appinteractive appinteractive merged commit b379f53 into develop Jun 30, 2018
@appinteractive appinteractive deleted the local_nyc branch July 8, 2018 20:01
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.

2 participants