-
-
Notifications
You must be signed in to change notification settings - Fork 10.2k
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
feature:document support multiple version #4452
Conversation
Signed-off-by: jianfei.zhang [email protected]
Signed-off-by: jianfei.zhang [email protected]
Signed-off-by: jianfei.zhang <[email protected]>
Signed-off-by: jianfei.zhang <[email protected]>
Signed-off-by: jianfei.zhang <[email protected]>
Codecov Report
@@ Coverage Diff @@
## master #4452 +/- ##
============================================
+ Coverage 53.54% 53.57% +0.03%
Complexity 2692 2692
============================================
Files 488 488
Lines 15229 15229
Branches 1587 1587
============================================
+ Hits 8154 8159 +5
+ Misses 6518 6514 -4
+ Partials 557 556 -1
Continue to review full report at Codecov.
|
Signed-off-by: jianfei.zhang <[email protected]>
What kind of version style do you want? |
Currently, apolloconfig.com refers to the latest version of the docs folder. |
Maybe we need to create version folder in docs and to put different versions of the files in the corresponding version folder. |
Currently, this seems the way to go. @Anilople do you have any suggestions? |
That will consume large cost to maintain it. Reference to different tag seems better. Can we combine them together? i.e generate the version folder during CI rather than commit them to the git repo directly. |
It will be best to do so during CI as this is the most consistent way. |
What should I do if we generate the version folder during CI? i don't kown how to do that |
There is a proposal in docsify that apollo uses to render the sites: docsifyjs/docsify#1289. |
What's the purpose of this PR
document support multiple version
Which issue(s) this PR fixes:
Fixes #4327