-
Notifications
You must be signed in to change notification settings - Fork 4.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: refine getting started (close #2111) #2281
Conversation
Looks good. What about finishing the zh translation |
BTW, I think 2-spaces indent is better |
Thanks for taking a look! @meteorlxy Finished Chinese translation. Just tried 2-spaces indent, but it can't pass lint
|
(alright, not familiar with remark-lint) |
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.
LGTM
cc @bencodezen |
Looks great to me. Thanks @ludanxer for your PR! |
* docs: getting started steps * docs: dir structure and basic config * docs: ready for review * docs: tweaks * docs: fix list-item-content-indent * docs: zh translation
Summary
Hi guys, I wrote a draft to close #2111 , what do you think?
I took the liberty of not recommend globally install VuePress, hope it's expected
What kind of change does this PR introduce? (check at least one)
If changing the UI of default theme, please provide the before/after screenshot:
Does this PR introduce a breaking change? (check one)
If yes, please describe the impact and migration path for existing applications:
The PR fulfills these requirements:
fix #xxx[,#xxx]
, where "xxx" is the issue number)You have tested in the following browsers: (Providing a detailed version will be better.)
If adding a new feature, the PR's description includes:
To avoid wasting your time, it's best to open a feature request issue first and wait for approval before working on it.
Other information: