-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
docs:upgrade docusaurus version #8770
Conversation
docs-website/yarn.lock
Outdated
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 like some of the docsearch and algolia packages didn't get upgraded
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.
🤦♀️ my bad. just manaually updated missing packages & regenerated yarn.lock again. Could you check if this is okay?
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
docs-website/yarn.lock
Outdated
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
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.
@yoonhyejin looks like the build is failing
docs-website/package.json
Outdated
@@ -39,7 +39,8 @@ | |||
"markprompt": "^0.1.7", | |||
"react": "^18.2.0", | |||
"react-dom": "18.2.0", | |||
"sass": "^1.43.2" | |||
"sass": "^1.43.2", | |||
"ts-node-dev": "^2.0.0" |
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.
dont think this new dep is necessary
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.
Yup just updated generateDocsDir for the newer version of ts-node
Checklist
Had to change some imports in Searchbar/index.js due to this change : https://github.com/facebook/docusaurus/pull/8757/files