-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
promote install instructions to top level menu item, enable mkdocs gh action #1681
Conversation
Welcome @geoffcline! |
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: geoffcline The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
draft new page visible here https://geoffcline.github.io/aws-load-balancer-controller/install/installation/ |
ready for review @M00nF1sh :) |
.github/workflows/mkdocs-deploy.yml
Outdated
@@ -0,0 +1,32 @@ | |||
# GCLINE@ 23NOV2020 |
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.
guess this should be amazon's copy right or leave empty :D
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 meant that more as a "who wrote this" but I can see the confusion, I'll fix
.github/workflows/mkdocs-deploy.yml
Outdated
- name: Checkout master | ||
uses: actions/checkout@v2 | ||
|
||
- name: Deploy 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.
we are using mike to deploy our multi-version docs.
I think we need to have our own actions or update mhausenblas/mkdocs-deploy-gh-pages
to support mike.
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'll take a look at how difficult that would be ... hopefully we can still do CD
how about the following layout
nice gh actions CI btw 👍 |
ok, I can work using that layout tomorrow 👍 |
Ok @M00nF1sh - I refactored based on your suggested nav. I also removed the github action, until I can get an action for mike working. Thanks! |
/lgtm..would you help rebase this. |
f424d6f
to
aaffaef
Compare
refactor to add deploy and examples top level nav
/lgtm |
… action (kubernetes-sigs#1681) * refactor install instructions, add new top tab refactor to add deploy and examples top level nav * move install materials back * move how-it-works under home * update docs to ref v2.1.0 branch Co-authored-by: Yang Yang <yyyng@amazon.com>
No description provided.