Update dependencies && remove unused vars #704
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
pretty huge pr here, but i promise that all i've done is
npx npm-check-updates --interactive --format group
to check)react-markdown
dependencyno-unused-vars
, and some other eslint issues.create-react-app
.create-react-app
, we cannot fix it properly.i did set out to solve all of the
npm i
issues, but it's not possible. the dependencies that baseui pull in do not support react 18, so in order to fix those problems we'll have to toss out all of baseui.so generally what i've learned is, if we want to use react 18:
both of these points involve some difficult choices and potentially lots of work.