-
Notifications
You must be signed in to change notification settings - Fork 49
NO-JIRA: use go workspaces #358
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
Changes from 5 commits
cb1f3ae
fe7eb7e
33761f1
fd1aba5
4c26a37
7ebe65c
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,13 @@ | ||
| go 1.24.0 | ||
|
|
||
| use ( | ||
| . | ||
| ./e2e | ||
| ./hack/tools | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Not sure if other's have opinions on this, but I think I'd prefer to not add hack/tools because that can contain/result in dependencies we don't want to have or may be incompatible at some point.. On the other hand: having it could force us to keep these tools also up to date and/or CVE free in some way. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. we could use |
||
| ./manifests-gen | ||
| ) | ||
|
|
||
| replace ( | ||
| github.com/metal3-io/baremetal-operator => github.com/metal3-io/baremetal-operator v0.5.1 | ||
| sigs.k8s.io/cluster-api => sigs.k8s.io/cluster-api v1.10.4 | ||
| ) | ||
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.
I can't find where this is used
I checked here on the
verify-depsjob config for ccapio, but that doesn't seem to be using this make targetThere 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.
for make target , it seems harmless just that it will call again ./hack/vendor.sh , just more intuitive..?
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.
@damdo the
verify-depsjob is a special one from ART, that ensures konflux / brew can build the repo (e.g with hermetic builds). We shouldnt touch it.https://steps.ci.openshift.org/reference/go-verify-deps
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.
Thanks @theobarberbany
Uh oh!
There was an error while loading. Please reload this page.
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.
@theobarberbany I was not implying we should change the
verify-depsone :)I was just following the above comment and not undestanding what it meant
If we know that this alias is not used by anything and is just an alias I suggest we drop it