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

Issue 22 add jest #29

Merged
merged 4 commits into from
Dec 19, 2019
Merged

Issue 22 add jest #29

merged 4 commits into from
Dec 19, 2019

Conversation

ao10
Copy link
Contributor

@ao10 ao10 commented Dec 16, 2019

Description

This PR adds Jest to our dev dependencies.

Related PRs

List related PRs against other branches:

branch PR
other_pr_description #

Related Issues

List related Issues:

|#22 |

Todos

  • Tests
  • Documentation

Deploy Notes

Notes regarding deployment the contained body of work.

Steps to Test or Reproduce

Outline the steps to test or reproduce the PR here.

npm run test

Impacted Areas in Application

List general components of the application that this PR will affect:

  • Testing

@ao10 ao10 requested a review from revjtanton December 16, 2019 07:54
Copy link
Member

@revjtanton revjtanton left a comment

Choose a reason for hiding this comment

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

I merged in master and fixed merge conflicts with `package-lock.json.

@revjtanton revjtanton merged commit 852b439 into master Dec 19, 2019
@revjtanton revjtanton deleted the issue-22-add-jest branch December 19, 2019 13:44
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

2 participants