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

chore: update deps #339

Closed
wants to merge 4 commits into from
Closed

chore: update deps #339

wants to merge 4 commits into from

Conversation

ethanwu10
Copy link
Member

@ethanwu10 ethanwu10 commented Feb 4, 2023

Highlights:

  • Gatsby 5
    • minimum node version is now node 18

Manual changes:
- Fix gatsby-env.d.ts (`declare` not valid in declaration context)
- Patch @chakra-ui/theme for chakra-ui/chakra-ui#7116

Typescript still has errors in libs; one should be fixed by Gatsby 5,
the other is a missing dependency from Chakra that needs to get dealt
with sometime later.
Notable changes:
- openapi-typescript dropped Prettier integration, so run prettier
  ourselves in the script
- workaround some weirdness in chakra - dependency override + reexport
  in theme file (gatsby plugin override file)

Remaining type errors in Gatsby (seem to be unreported as of the current
moment)
Gatsby 5 requires Node 18; update docs and CI
@ocfjenkins
Copy link

ocfjenkins bot commented Feb 4, 2023

View the preview deployment of this pull request here: https://PR-339.new.ocf.berkeley.edu

@ben9583
Copy link
Member

ben9583 commented Oct 5, 2023

Closing because #347 was pulled instead

@ben9583 ben9583 closed this Oct 5, 2023
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