Skip to content

Conversation

@ahmetcetin
Copy link

closes #18

Copy link
Owner

@nanomosfet nanomosfet left a comment

Choose a reason for hiding this comment

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

Thank you for contributing!
We should also note that the build should pass, could you add that in here?

could you add how to run tests / lint etc.

@nanomosfet
Copy link
Owner

I also need to check why the build is failing. it is related to node v13

requested changes

* Lint: npm run lint
* Test: npm run test
* Test coverage: npm run coveralls
Copy link
Owner

Choose a reason for hiding this comment

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

this is actually used for CI

you can do:

Suggested change
* Test coverage: npm run coveralls
* Test coverage: npm test -- --coverage

or add script with "coverage" : "jest --coverage"

and make this

Suggested change
* Test coverage: npm run coveralls
* Test coverage: npm run coverage

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.

Create contributing.md

2 participants