-
Notifications
You must be signed in to change notification settings - Fork 367
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
POC: [M3-7538] - Cloud Manager GH Pages Docs #9960
Conversation
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 awesome! Love that we can use Github Pages for this!
Is the choice of Jekyll just because Github supports it without needing a dedicated Github Action to build and publish? I like how it's super built in, but the Gemfiles, html, and scss will definitely placebo me into feeling our repo is bloated and messy.
What do you think about using something more modern and applicable to our Typescript ecosystem like https://vitepress.dev. I'm not super familiar with Vitepress and how it compares to jekyll, but it does seem to be the standard for the Typescript ecosystem as of now. I guess a disadvantage would be that we need a Github Action with a build step and more node_modules?
Coverage Report: ✅ |
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 a big fan of Github Pages, thanks for setting this up!
I've left some nitpicks on typos and wording that I found as I read through the docs. As far as the actual pages site, I think it looks nice. Good choice on theme; I love the search. My one complaint so far about layout is that the side menu seems really wide. Can we we adjust that? I was going to play with the theme overrides a bit to see what we've got. Either way, not a blocker.
|
||
This doc builder uses default GitHub pages and its Jekyll site builder to deploy documentation pages to [https://linode.github.io/manager](https://linode.github.io/manager). | ||
|
||
It automatically get deployed when we release to `master`. |
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.
It automatically get deployed when we release to `master`. | |
It automatically gets deployed when we release to `master`. |
|
||
In order to develop locally, start by setting up your tolling locally by following the steps on this [page](https://docs.github.com/en/pages/setting-up-a-github-pages-site-with-jekyll/testing-your-github-pages-site-locally-with-jekyll). | ||
|
||
This should be done from the /docs/ directory. |
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 should be done from the /docs/ directory. | |
This should be done from the `/docs/` directory. |
- modify the styles by updating `_sass/color_schemes/akamai.scss`. | ||
- modify layouts by adding/updating HTML templates to the `_includes` directory. | ||
|
||
See [https://just-the-docs.com/docs/customization](https://just-the-docs.com/docs/customization) for mor info. |
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.
See [https://just-the-docs.com/docs/customization](https://just-the-docs.com/docs/customization) for mor info. | |
See [https://just-the-docs.com/docs/customization](https://just-the-docs.com/docs/customization) for more info. |
"@linode/manager": Added | ||
--- | ||
|
||
Cloud Manager Documenation Builder for Gihub Pages ([#9960](https://github.com/linode/manager/pull/9960)) |
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.
Cloud Manager Documenation Builder for Gihub Pages ([#9960](https://github.com/linode/manager/pull/9960)) | |
Cloud Manager Documentation Builder for Github Pages ([#9960](https://github.com/linode/manager/pull/9960)) |
|
||
## Get Started | ||
|
||
To setup cloud manager and get ready for development, process to the [get started docs](GETTING_STARTED.md). |
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.
To setup cloud manager and get ready for development, process to the [get started docs](GETTING_STARTED.md). | |
To setup Cloud Manager and get ready for development, proceed to the [getting started docs](GETTING_STARTED.md). |
|
||
Find resources about specific areas of development in Akamai's Cloud Manager. | ||
|
||
For any other issues feel free to open an issue to report a bug or request a feature. |
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.
For any other issues feel free to open an issue to report a bug or request a feature. | |
For any other issues, feel free to [open a GitHub Issue](https://github.com/linode/manager/issues/new) to report a bug or request a feature. |
@bnussman-akamai I was waiting for this comment 😄 yes, it is what GH pages using under the hood. The perception that the repo is "bloated" is quite subjective and personal, and I'd like to make the debate more objective. The quest for a 100% TS repo does not convey much to me. I actually like the idea of diversifying the tech and focus on the right tool for the job. In other words I don't really care that much what brand is the guitar, I care about the music it makes 🎵. That being I am open to other options (I am sure Vitpress is fine and is a proper tool for it) but it seems overkill to me to spend a lot more time to achieve the exact same (or similar) result? This was build in half a day so there is that. Curious to hear other opinions as well. |
|
||
# Development Guide | ||
|
||
Find resources about specific areas of development in Akamai's Cloud Manager. |
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.
Do you mean the documentation? If so should we point them here https://www.linode.com/docs/?
I think both solutions are valid. But given that this was an easy and fast setup and it does everything we want right now, my vote is to keep it. In the future, if we want more features, it seems like vitepress is a good option. And we wouldnt have to tear down much to make the transition. |
Also, might be worth adding |
I agree having a 100% TS repo doesn't mean much on its own, but one factor to consider is the skillset of the team and whether or not we are in a position to support this little slice of Ruby. @abailly-akamai do you have a sense for the likelihood of ever having to run/debug this locally? Maybe a good cafe topic to see if we have Ruby skills on the team (and/or interest in learning). |
@johnwcallahan yup am planning to bring this to the cafe. I don't feel that strong about it, but i also think that apart from installing tools and dependencies the ruby factor is a tad irrelevant cause it's just supporting a compiler in this case. With a bit of work we could probably move this to a GH action as well and remove this from the codebase. I wrote steps to run this locally as part of this PR but let's remember this is POC, showing what the end result is, we can find ways to clean it up - what I want to be cautious about is jump on the latest coolest JS framework bandwagon that's going to require pouring gallons of JS to build < 20 pages of docs. |
|
||
## Local Development | ||
|
||
In order to develop locally, start by setting up your tolling locally by following the steps on this [page](https://docs.github.com/en/pages/setting-up-a-github-pages-site-with-jekyll/testing-your-github-pages-site-locally-with-jekyll). |
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 you meant "tooling" instead of "tolling".
This is a great idea! Upon initial review of the UI/UX I have the following recommendations:
|
Description 📝
This PR is a rather quick attempt at configuring a basic Jekyll site builder for our docs and publish it automatically when pushed to a branch of our choice (in this case
M3-7538
, hence why this branch isn't from my fork.The advantage of this approach is to integrate things easily with very minimal dependencies and get dedicated hosting for the static site. It could be done in many ways via other static builders (hugo etc) but I don't think we need to go too far at the moment since this is very much a built in, secure solution we can just use almost out of the box.
I also used a theme that had a lot of decent features (navigation, search, responsive design, easy to customize) to facilitate development and user experience. The search is really super useful! 🎉
The site is published at https://linode.github.io/manager
In this PR i did not update the main README to link to this doc site since it may not be ready for prime time but that can easily be done now or later, depending on feedback for the POC.
NOTE: some doc pages may have formatting issues (unrelated to this PR) which I will update if this PR moves forward.
Changes 🔄
Preview 📷
How to test 🧪
Verification steps
As an Author I have considered 🤔
Check all that apply