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 testing setup to use jest #53

Merged
merged 13 commits into from
Oct 21, 2020
Merged

Conversation

lifeart
Copy link
Contributor

@lifeart lifeart commented Sep 26, 2020

@lifeart lifeart changed the title add jest replace mocha to jest Sep 26, 2020
@lifeart lifeart marked this pull request as ready for review September 26, 2020 12:34
@lifeart lifeart changed the title replace mocha to jest replace mocha -> jest (+eslint) Sep 26, 2020
@lifeart lifeart changed the title replace mocha -> jest (+eslint) 1/3 replace mocha -> jest (+eslint) Sep 26, 2020
@lifeart lifeart mentioned this pull request Sep 26, 2020
Copy link
Collaborator

@simonihmig simonihmig left a comment

Choose a reason for hiding this comment

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

LGTM, just a minor remark!

lib/summarize.js Outdated Show resolved Hide resolved
Copy link
Collaborator

@simonihmig simonihmig left a comment

Choose a reason for hiding this comment

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

LGTM, thank you @lifeart! (didn't notice you already updated the PR according to my feedback, sorry for the delay!)

@stefanpenner do yo want to have a second look? Otherwise I would gladly merge this.

@lifeart
Copy link
Contributor Author

lifeart commented Oct 19, 2020

UP
cc @stefanpenner

@simonihmig simonihmig changed the title 1/3 replace mocha -> jest (+eslint) Update testing setup to use jest Oct 21, 2020
@simonihmig simonihmig merged commit 432d2a9 into stefanpenner:master Oct 21, 2020
@simonihmig
Copy link
Collaborator

@lifeart merged this! Can you please rebase your next PR, so we get a clean diff / commit history? Thanks again for your work! 👍

@lifeart lifeart deleted the add-jest branch October 21, 2020 08:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants