Skip to content

Comments

Added install dependencies & tests setup#33

Merged
ma-oliveiramarques merged 2 commits intomasterfrom
tests-documentation
May 2, 2022
Merged

Added install dependencies & tests setup#33
ma-oliveiramarques merged 2 commits intomasterfrom
tests-documentation

Conversation

@ma-oliveiramarques
Copy link
Contributor

Attempt to document tests && improve readme file.
Close #15

README.md Outdated
```
Jest
```bash
npm install --save-dev jest
Copy link
Member

Choose a reason for hiding this comment

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

This is unnecessary because the one who clone this repo just need to run npm install or npm i to install all the dependencies present in the package.json file

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right! Should I keep the npm test??

Copy link
Member

Choose a reason for hiding this comment

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

Right! Should I keep the npm test??

Yes because someone who clone this repo should run the tests in order to prove or just to see if the tests pass.

@ma-oliveiramarques ma-oliveiramarques merged commit f4775e0 into master May 2, 2022
@ma-oliveiramarques ma-oliveiramarques deleted the tests-documentation branch May 2, 2022 15:59
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.

Document unit tests

2 participants