-
Notifications
You must be signed in to change notification settings - Fork 189
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
Fixes #2792 Set up Docusaurus website #2853
Conversation
4c749d8
to
7ff60eb
Compare
Requesting reviews from maintainers who have commented and reviewed the previous PR :) @humphd @JerryHue @aserputov @Kevan-Y @manekenpix and @sirinoks because I know how much she loves docs |
7ff60eb
to
4cf78ae
Compare
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.
I'm excited for Docusaurus being set up and organize all the docs related issues we have!
src/docs/package.json | ||
src/docs/package-lock.json |
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.
I think these might be unnecessary, since we already have package.json
and package-lock.json
in the list, and the .gitignore
rules are the same for .prettierignore.
This is more of a nitpick, so it might not even be necessary to change, just letting you know.
@cindyledev , created a follow-up issue to dockerize Docusaurus service. #2865 |
@cindyledev, we just need to rebase + squash the commits and we can merge this. |
4cf78ae
to
119709d
Compare
Issue This PR Addresses
Fixes #2792
Fixes #2746
Fixes #290
Type of Change
Description
Steps to test the PR
This PR does the following:
.eslintrc.js
To test this PR locally:
git remote add cindyledev https://github.com/cindyledev/telescope.git
git fetch cindyledev
git checkout issue-2792
pnpm install
cd src/docs
pnpm start
Checklist