-
Notifications
You must be signed in to change notification settings - Fork 239
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
feat: upgrade to react 18 #1663
Conversation
32cb867
to
6769f69
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1663 +/- ##
==========================================
+ Coverage 90.09% 90.22% +0.13%
==========================================
Files 338 338
Lines 5784 5800 +16
Branches 1370 1410 +40
==========================================
+ Hits 5211 5233 +22
+ Misses 557 550 -7
- Partials 16 17 +1 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
6769f69
to
0111b2d
Compare
@@ -341,7 +341,7 @@ class Calculator extends Component { | |||
<li>arccsch(4x+y)</li> | |||
</ul> | |||
</td> | |||
<td dir="auto" /> | |||
<td aria-label="todo: what is this? it's causing a jsx-a11y/control-has-associated-label error" dir="auto" /> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
todo: figure out what's up with this
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sandbox deployment successful 🚀 |
0111b2d
to
e1c9149
Compare
e1c9149
to
ea4d302
Compare
Sandbox deployment successful 🚀 |
Sandbox deployment successful 🚀 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good! 👍🏼
Part 2 of #1587