Skip to content

Update README.md - #1

Open
daaain wants to merge 1 commit into
technosophos:masterfrom
daaain:master
Open

daaain wants to merge 1 commit into
technosophos:masterfrom
daaain:master

Conversation

@daaain

@daaain daaain commented Aug 28, 2020

Copy link
Copy Markdown

I got tripped up by two important differences between the README and how things behave:

  1. Github Pages are now on github.io (not github.com)
  2. Helm expects the directory name where the chart is (so alpine and not mychart here)

@manojtr

manojtr commented Dec 11, 2020

Copy link
Copy Markdown

I got tripped up by two important differences between the README and how things behave:

  1. Github Pages are now on github.io (not github.com)
  2. Helm expects the directory name where the chart is (so alpine and not mychart here)

@daaain
Can we do this for any git repo or just with github? I tried the same with a gitlab repo and doesn't work. Did it work for you?

@daaain

daaain commented Dec 11, 2020

Copy link
Copy Markdown
Author

Can we do this for any git repo or just with github? I tried the same with a gitlab repo and doesn't work. Did it work for you?

I don't know much about working with Gitlab, but they do also seem to have Pages:
https://docs.gitlab.com/ee/user/project/pages/

But even if that didn't work, you can also deploy to a (free) static host like Netlify, Firebase Hosting, etc with a few lines of CI script.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants