-
Notifications
You must be signed in to change notification settings - Fork 512
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: Move architecture components doc to Hugo bundles #1517
Conversation
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
Signed-off-by: Marco Pracucci <marco@pracucci.com>
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.
This looks good to me!
Again, I have run the local webserver and all images look good and pages ToC is as expected.
Makefile
Outdated
@@ -502,4 +502,4 @@ integration-tests: cmd/mimir/$(UPTODATE) | |||
|
|||
include docs/docs.mk | |||
DOCS_DIR = docs/sources | |||
docs: doc | |||
docs: |
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 guess this prerequisite was more annoying than useful?
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.
This is an unintended change :( Sorry for spotting it! Reverted.
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.
No problem at all. I think there is certainly a case to be made that it gets in the way for rapid iteration on docs. If it's useful, I cd to docs
and run make docs
there when I know that I don't need to rebuild any of the documentation. The Makefile in that directory does not have the prerequisite target doc
so just runs the webserver.
Signed-off-by: Marco Pracucci <marco@pracucci.com>
* Migrated query-frontend doc to Hugo bundle Signed-off-by: Marco Pracucci <marco@pracucci.com> * Migrated query-scheduler doc to Hugo bundle Signed-off-by: Marco Pracucci <marco@pracucci.com> * Migrated compactor doc to Hugo bundle Signed-off-by: Marco Pracucci <marco@pracucci.com> * Reverted unintended change in Makefile Signed-off-by: Marco Pracucci <marco@pracucci.com>
What this PR does
In this PR I'm moving components doc which has images to Hugo bundles.
Which issue(s) this PR fixes or relates to
Part of #1497.
Checklist
CHANGELOG.md
updated - the order of entries should be[CHANGE]
,[FEATURE]
,[ENHANCEMENT]
,[BUGFIX]