-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Updated Readme.md #9647
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
Updated Readme.md #9647
Conversation
Changed the wording to point to the default installation method we want people to use.
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 good. Go for it! 🚀
/werft run |
/werft run 👍 started the job as gitpod-build-lucasvaltl-installer-readme-update.1 |
@lucasvaltl Sorry, had to move back to draft bc it was blocking the merge queue. 😕 |
Wanted to do this as well right now. Thanks Gero! 🙏 @lucasvaltl I guess you need to rebase this PR anyways to fix the build error as soon as #9648 is in. |
@lucasvaltl I confirm that rebasing the branch will solve the problem of Werft failing to build. |
Where we need it, take it from the environment.
This was previously hard-coded to false in the server configmap.
This was previously hard-coded to true in the server configmap.
This was previously hard-coded to empty in the server configmap.
Extract named structs for fields that were nested under `ServerConfig` so that they can be instantiated in tests.
Eeeeh I tried rebasing, let's just say I'm just going to start with a fresh PR :D Feels like that is easier. Closing this and restarting this :) |
---> trying again here #9658 |
Description
Changed the wording to point to the default installation method we want people to use.
Related to https://github.com/gitpod-io/website/pull/1975
Release Notes