Skip to content

Replace Karma with Jest, run tests on CI (#87)#101

Merged
adamkleingit merged 2 commits into
mobxjs:masterfrom
kubk:jest
Dec 9, 2019
Merged

Replace Karma with Jest, run tests on CI (#87)#101
adamkleingit merged 2 commits into
mobxjs:masterfrom
kubk:jest

Conversation

@kubk
Copy link
Copy Markdown
Contributor

@kubk kubk commented Dec 8, 2019

Related issue: #87

  • Karma test runner was replaced with Jest
  • Fixed lint errors in ng2-mobx.spec.ts
  • Added tests to CI
@adamkleingit adamkleingit merged commit 07b0b79 into mobxjs:master Dec 9, 2019
@kubk
Copy link
Copy Markdown
Contributor Author

kubk commented Dec 9, 2019

Hi @adamkleingit What do you think about running Prettier across entire project? Currently tslint is complaining about double quotes, that's why I replaced them with single quotes. Can make a PR. Thank you.

@adamkleingit
Copy link
Copy Markdown
Member

Yeah I'm all for it, thanks! We should use husky and lint-staged to ensure it runs on commit automatically.
Here is an example from our company's boilerplate:
https://bitbucket.org/500tech/react-redux-project-boilerplate/src/master/package.json

@kubk kubk mentioned this pull request Jan 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants