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

Update Javascript version for TripService #58

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

udebella
Copy link

@udebella udebella commented Oct 5, 2020

Scope of the changes :

  • Updated dependencies to be up-to-date with last versions.
  • Added NVM (node version manager) to specify node version to use when working on the kata
  • Updated code coverage tool to generate coverage report each time we run the test in /coverage folder
  • With the last version of node, introduced es6 modules inside the project
  • Used Chai as assertion library which is more user friendly than the one bundled in Nodejs.
  • Lastly, fixed the user class that was not properly implemented (it is correctly implemented in other languages of this kata)

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

1 participant