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(deps): upgrade dependencies #4166

Merged
merged 7 commits into from
Sep 9, 2024
Merged

chore(deps): upgrade dependencies #4166

merged 7 commits into from
Sep 9, 2024

Conversation

yaacovCR
Copy link
Contributor

Errata:

Left typescript at v5.4.x as docusaurus currently conflicts with typescript v5.5.x. Left eslint v8.x as not all plugins are compatible with the new flat file config format introduced in eslint v9.x. Left at docusaurus 3.4.x as docusaurus-plugin-typedoc-api has a bug with docusaurus v3.5.x.

Code changes, in no particular order:

  1. Prettier formatting changes.
  2. Prettier moved to an async API, but the writeGeneratedFile utility, which previously included prettifying was passed as a callback function to TS and had to stay sync, so prettifying was separated into a separate async function -- the callback function luckily did not seem to actually requiring another round of prettifying, as it just involved renaming. All the other callsites of the new utility had to be made async. In the alternate, I investigated @prettier/sync and the lower-level make-synchronized and make-synchronous packages, but I could not get them working.
  3. Plenty of eslint rule changes! I have tried to make sure that the rule list orders now match the linked documentation, so that further updates might be easier.
  4. Minor docusaurus config tweaks to get the build to pass.

@yaacovCR yaacovCR requested a review from a team as a code owner August 16, 2024 05:26
Copy link

Hi @yaacovCR, I'm @github-actions bot happy to help you with this PR 👋

Supported commands

Please post this commands in separate comments and only one per comment:

  • @github-actions run-benchmark - Run benchmark comparing base and merge commits for this PR
  • @github-actions publish-pr-on-npm - Build package from this PR and publish it on NPM

Copy link

netlify bot commented Aug 16, 2024

Deploy Preview for compassionate-pike-271cb3 ready!

Name Link
🔨 Latest commit 4e76da0
🔍 Latest deploy log https://app.netlify.com/sites/compassionate-pike-271cb3/deploys/66de6aff14dbfc000874bb3e
😎 Deploy Preview https://deploy-preview-4166--compassionate-pike-271cb3.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JoviDeCroock
Copy link
Member

@yaacovCR can we hold off on this until we merge #4165 due to the many new conflicts that will arise.

@yaacovCR
Copy link
Contributor Author

yaacovCR commented Sep 7, 2024

@JoviDeCroock --

I think we can merge this now that #4171 has landed? If we do decide to go forward with #4165, my understanding is that there are no actual code changes any more from 16.x.x that will need to land, and this would only be for making the git history look pretty, so the conflicts at this point are moot?

@yaacovCR
Copy link
Contributor Author

yaacovCR commented Sep 8, 2024

Updated again, this time docusaurus-plugin-typedoc-api has been upgraded to work with latest docusaurus, so I updated both of those, and I noticed that we can actually upgrade to typescript 5.5 as we get some warnings from underlying typedoc used by plugin, but website still compiles fine.

So everything upgraded except eslint, not all of our plugins support v9 even with new flat config file format disabled.

@yaacovCR yaacovCR added the PR: polish 💅 PR doesn't change public API or any observed behaviour label Sep 8, 2024
Errata:

Left typescript at v5.4.x as docusaurus currently conflicts with typescript v5.5.x.
Left eslint v8.x as not all plugins are compatible with the new flat file config format introduced in eslint v9.x.
Left at docusaurus 3.4.x as docusaurus-plugin-typedoc-api has a bug with docusaurus v3.5.x.

Code changes, in no particular order:

1. Prettier formatting changes.
2. Prettier moved to an async API, but the `writeGeneratedFile` utility, which previously included prettifying was passed as a callback function to TS and had to stay sync, so prettifying was separated into a separate async function -- the callback function luckily did not seem to actually requiring another round of prettifying, as it just involved renaming. All the other callsites of the new utility had to be made async. In the alternate, I investigated @prettier/sync and the lower-level make-synchronized and make-synchronous packages, but I could not get them working.
3. Plenty of eslint rule changes! I have tried to make sure that the rule list orders now match the linked documentation, so that further updates might be easier.
4. Minor docusaurus config tweaks to get the build to pass.
this time `docusaurus-plugin-typedoc-api` has been upgraded to work with latest `docusaurus`, so I updated both of those, and I noticed that we can actually upgrade to typescript 5.5 as we get some warnings from underlying typedoc used by plugin, but website still compiles fine.

So everything upgraded except eslint, not all of our plugins support v9 even with new flat config file format disabled.
@yaacovCR yaacovCR force-pushed the update branch 3 times, most recently from 5a17e16 to 8137767 Compare September 8, 2024 20:24
@yaacovCR
Copy link
Contributor Author

yaacovCR commented Sep 8, 2024

Seems like our ci does not like typescript 5.5, may have to roll that back

@yaacovCR yaacovCR merged commit 6c56e05 into graphql:main Sep 9, 2024
20 checks passed
@yaacovCR yaacovCR deleted the update branch September 9, 2024 10:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
PR: polish 💅 PR doesn't change public API or any observed behaviour
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants