-
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
overview: warn about install-config.yaml deletion #1122
Conversation
Motivation for the removal and further discussion in #898. |
@@ -60,7 +60,7 @@ mv initial/install-config.yaml . | |||
rm -rf initial |
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.
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.
.openshift_install_state.json
Can you also squash down to one commit? |
Does GitHub GUI allow this basic functionality? I'll use the command line if not. |
Not to my knowledge. |
Aha, looks like you've allowed maintainer edits, because GitHub let me force-push a squash to your branch with 2b2a6a0 -> 4c6fca4. Does 4c6fca4 look good to you, @dankenigsberg ? |
lvgtm. |
/lgtm Thanks :) |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: dankenigsberg, wking The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
I don't understand why it makes sense and find it surprising. Let's warn other readers.
@crawford can you confirm that this is the intended behavior, and maybe explain the logic behind it?