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

Package update #17

Merged
merged 4 commits into from
Mar 22, 2023
Merged

Package update #17

merged 4 commits into from
Mar 22, 2023

Conversation

Omarley7
Copy link
Owner

Going though packages to check what is stopping the unit tests from working properly.

  • Removed babel, as jest is using ts-jest instead.
  • Babel can later be added if transpiling between different js modules is needed. (Extended browser support).
  • Removed @jest/gloabls since it was conflicting with dom Matchers.
  • Added import to all test files.

Added types for react and react-dom.
Removed conflicting @jest/globals package
Removed all babel, jest is using ts-jest.
It's no longer needed in tests.
@Omarley7 Omarley7 self-assigned this Mar 21, 2023
Re-instaded babel needed for react-scripts to run.
Removed transform from jest config and eslint.
client/.eslintrc Show resolved Hide resolved
Slight modification to eslint.
@codeclimate
Copy link

codeclimate bot commented Mar 22, 2023

Code Climate has analyzed commit cec4168 and detected 0 issues on this pull request.

View more on Code Climate.

@SyntaxXeror SyntaxXeror merged commit 0a3c3f2 into feature/distributed-demo Mar 22, 2023
@SyntaxXeror SyntaxXeror deleted the package-update branch March 22, 2023 09:59
Omarley7 added a commit that referenced this pull request Mar 22, 2023
* Updated typescript and jest libaries.
Added types for react and react-dom.
Removed conflicting @jest/globals package
Removed all babel, jest is using ts-jest.

* Added import of jest-dom to jest.config.json.
It's no longer needed in tests.

* Build with no warnings and functional test.

Re-instaded babel needed for react-scripts to run.
Removed transform from jest config and eslint.

* Remved jest/globals reference from tests.
Slight modification to eslint.
Omarley7 added a commit that referenced this pull request Mar 29, 2023
* Updated typescript and jest libaries.
Added types for react and react-dom.
Removed conflicting @jest/globals package
Removed all babel, jest is using ts-jest.

* Added import of jest-dom to jest.config.json.
It's no longer needed in tests.

* Build with no warnings and functional test.

Re-instaded babel needed for react-scripts to run.
Removed transform from jest config and eslint.

* Remved jest/globals reference from tests.
Slight modification to eslint.
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