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

fix: loosen react dependency versions #78

Merged
merged 3 commits into from
Oct 3, 2022
Merged

fix: loosen react dependency versions #78

merged 3 commits into from
Oct 3, 2022

Conversation

SimeonC
Copy link
Collaborator

@SimeonC SimeonC commented Sep 30, 2022

There are probably other changes that need to be taken to support react 18 fully, but most of it should work with current setup. Other changes will be done in next branch but this at least allows us to try out react 18 without the CI failing.

Specifically related to the TableKit next branch which has some installation issues due to this - canary version was tested here: tablecheck/tablekit#131

πŸ“¦ Published PR as canary version: Canary Versions

✨ Test out this PR locally via:

npm install @tablecheck/eslint-config@1.8.2-canary.78.3156990612.0
npm install @tablecheck/scripts@2.3.8-canary.78.3156990612.0
# or 
yarn add @tablecheck/eslint-config@1.8.2-canary.78.3156990612.0
yarn add @tablecheck/scripts@2.3.8-canary.78.3156990612.0

There are probably other changes that need to be taken to support react 18 fully, but most of it should work with current setup. Other changes will be done in next branch but this at least allows us to try out react 18 without the CI failing.
@SimeonC SimeonC added the patch Increment the patch version when merged label Sep 30, 2022
@github-actions
Copy link

github-actions bot commented Sep 30, 2022

Unit Test Results

βŸβ€„βŸβ€„3 filesβ€„β€ƒβŸβ€„12 suites   10s ⏱️
142 tests 142 βœ”οΈβ€ƒ0 πŸ’€β€ƒ0 ❌
156 runs  156 βœ”οΈβ€ƒ0 πŸ’€β€ƒ0 ❌

Results for commit 6e3f21f.

♻️ This comment has been updated with latest results.

@SimeonC SimeonC self-assigned this Sep 30, 2022
@SimeonC SimeonC enabled auto-merge (rebase) September 30, 2022 08:06
@SimeonC SimeonC merged commit 99e5ee7 into main Oct 3, 2022
@SimeonC SimeonC deleted the loosen-react-deps branch October 3, 2022 02:39
@tablecheck-public-automation
Copy link
Collaborator

πŸš€ PR was released in @tablecheck/eslint-config@1.8.2, @tablecheck/scripts@2.3.8 πŸš€

@tablecheck-public-automation tablecheck-public-automation added the released This issue/pull request has been released. label Oct 3, 2022
@tablecheck-public-automation
Copy link
Collaborator

πŸš€ PR was released in @tablecheck/eslint-config@1.8.2, @tablecheck/scripts@2.3.8 πŸš€

1 similar comment
@tablecheck-public-automation
Copy link
Collaborator

πŸš€ PR was released in @tablecheck/eslint-config@1.8.2, @tablecheck/scripts@2.3.8 πŸš€

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
patch Increment the patch version when merged released This issue/pull request has been released.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants