-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
move API types to consistent packages #14843
Conversation
Hey, only 20k lines. That's better than I thought it would be. |
@@ -0,0 +1,135 @@ | |||
#!/bin/bash |
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.
Are you expecting this to stick around?
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.
Are you expecting this to stick around?
Not in the sense of "this script needs to be pretty". I want the script in this pull so that people can see if it does roughly what they expect instead of reviewing the 20k diff.
|
a3a2017
to
c0cf79c
Compare
I'd like this after #14701 |
c0cf79c
to
a9d5891
Compare
re[test] |
534d7eb
to
a9d5891
Compare
a9d5891
to
af2e30d
Compare
failed to provision re[test] green except for gce, yay! |
Evaluated for origin test up to af2e30d |
[testextended][extended:core(builds)] |
@liggitt builds on the SCC pull. Can you take a look. This actually has a shot at being green and being on a valid base for a force. |
Evaluated for origin testextended up to af2e30d |
continuous-integration/openshift-jenkins/testextended Running (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin_extended/740/) (Base Commit: 07cba93) (PR Branch Commit: af2e30d) (Extended Tests: core(builds)) |
@@ -1 +0,0 @@ | |||
../main.go |
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.
the changes to this package surprised me
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.
the changes to this package surprised me
weird, but it looks like it will hold (symlink to file with same content) and be fixable immediately after.
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.
that's fine
LGTM |
looks good to merge on green tests against master HEAD |
continuous-integration/openshift-jenkins/test SUCCESS (https://ci.openshift.redhat.com/jenkins/job/test_pull_request_origin/2618/) (Base Commit: 07cba93) (PR Branch Commit: af2e30d) |
Base commit is the current HEAD of master (minus webconsole commits), tests are green, currently merging pull flaked, it won't get better than this. Merging and aborting the current job. |
@openshift/api-review as we discussed, this moves our API types to make them consistent with all other api types in existence. This has to merge for 3.6 or we will be unable to pick changes back.
[test] to see what was missed.