Skip to content

React18 #2504

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

Merged
merged 12 commits into from
Jun 22, 2023
Merged

React18 #2504

merged 12 commits into from
Jun 22, 2023

Conversation

chownces
Copy link
Contributor

@chownces chownces commented Jun 9, 2023

Description

Upgrade to react v18.

Blocked by the follow 2 changes:

Type of change

  • Dependency update

How to test

Checklist

  • I have tested this code
  • I have updated the documentation

@github-actions
Copy link

github-actions bot commented Jun 9, 2023

Pull Request Test Coverage Report for Build 5342448651

  • 9 of 14 (64.29%) changed or added relevant lines in 10 files are covered.
  • 3 unchanged lines in 2 files lost coverage.
  • Overall coverage decreased (-0.02%) to 36.473%

Changes Missing Coverage Covered Lines Changed/Added Lines %
src/pages/login/Login.tsx 1 2 50.0%
src/commons/editor/UseHighlighting.tsx 0 2 0.0%
src/index.tsx 0 2 0.0%
Files with Coverage Reduction New Missed Lines %
src/commons/utils/SortableList.tsx 1 0%
src/features/envVisualizer/EnvVisualizerLayout.tsx 2 66.35%
Totals Coverage Status
Change from base Build 5334031974: -0.02%
Covered Lines: 5266
Relevant Lines: 13580

💛 - Coveralls

@chownces chownces added the blocked Something else needs pass review first label Jun 19, 2023
@RichDom2185 RichDom2185 self-requested a review June 21, 2023 18:00
@chownces chownces removed the blocked Something else needs pass review first label Jun 22, 2023
@chownces chownces marked this pull request as ready for review June 22, 2023 09:20
Copy link
Member

@RichDom2185 RichDom2185 left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@RichDom2185 RichDom2185 merged commit 0ba9457 into master Jun 22, 2023
@RichDom2185 RichDom2185 deleted the react18 branch June 22, 2023 10:43
RichDom2185 pushed a commit to NUS-CS1101S/cadet-frontend that referenced this pull request Jul 15, 2023
* Upgrade React and corresponding types to latest version

* Deprecate usage of React.SFC

* Fix miscellaneous type errors

* Upgrade react-konva to fix type errors with react18

* Fix typings for react-sortable-hoc

* Shift root render to new API

* Fix @types/react versioning

* Fix navigation bar typing

* Fix useEffect return type

* Upgrade @testing-library/react and react-test-renderer to match react v18 and fix tests

* Workaround deprecated ReactDOM.render used by Blueprintjs v4

* Update notification import lines
RichDom2185 pushed a commit to NUS-CS1101S/cadet-frontend that referenced this pull request Aug 15, 2023
* Upgrade React and corresponding types to latest version

* Deprecate usage of React.SFC

* Fix miscellaneous type errors

* Upgrade react-konva to fix type errors with react18

* Fix typings for react-sortable-hoc

* Shift root render to new API

* Fix @types/react versioning

* Fix navigation bar typing

* Fix useEffect return type

* Upgrade @testing-library/react and react-test-renderer to match react v18 and fix tests

* Workaround deprecated ReactDOM.render used by Blueprintjs v4

* Update notification import lines
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.

2 participants