-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Migrate serving to use the knative.dev/serving import path #4521
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mattmoor 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 |
/lgtm I heartily endorse this PR. The overlapping of |
e17909e
to
c53304a
Compare
96baafb
to
c4eea68
Compare
c4eea68
to
34f3e2c
Compare
/hold cancel |
34f3e2c
to
f667360
Compare
``` sed -i '[email protected]/knative/[email protected]/serving@g' $(find -name '*.go' | xargs grep github.com/knative/serving | cut -d':' -f 1 | uniq) sed -i '[email protected]/knative/[email protected]/serving@g' $(find -name '*.yaml' | xargs grep github.com/knative/serving | cut -d':' -f 1 | uniq) ./hack/update-codegen.sh ```
f667360
to
0c94fbe
Compare
/retest |
/retest |
/test pull-knative-serving-go-coverage |
/test pull-knative-serving-integration-tests |
In PR knative#4521 we accidentally replaced all https://github.com/knative/serving/... links in our Go source files with https://knative.dev/serving/... Those links should have been left as github.com. This fixes that. The commit is entirely scripted via: sed -i 's@https://knative.dev/serving@https://github.com/knative/serving@g' $(find -name '*.go' | xargs grep knative.dev/serving | cut -d':' -f 1 | uniq)
In PR #4521 we accidentally replaced all https://github.com/knative/serving/... links in our Go source files with https://knative.dev/serving/... Those links should have been left as github.com. This fixes that. The commit is entirely scripted via: sed -i 's@https://knative.dev/serving@https://github.com/knative/serving@g' $(find -name '*.go' | xargs grep knative.dev/serving | cut -d':' -f 1 | uniq)
I have broken this into two commits:
This is a BREAKING change for all contributors
After this has merged, the following changes are needed:
knative/serving
directory to${GOPATH}/src/knative.dev/serving
In order to merge this, we are going to have to effectively take a lock on Prow for a few hours because we will need to change the repository configuration and land this change (which also won't pass CI until that change is made).
Fixes: #971
/hold
Until after we cut 0.7, and land basically every change that's ready to go.