Skip to content

Change gopath #178

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

Merged
2 commits merged into from Nov 25, 2022
Merged

Change gopath #178

2 commits merged into from Nov 25, 2022

Conversation

ghost
Copy link

@ghost ghost commented Nov 16, 2022

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
We refactor to use gopath instead.

Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #107

Special notes for your reviewer:

Please confirm that if this PR changes any image versions, then that's the sole change this PR makes.

TODOs:

  • squashed commits
  • includes documentation
  • adds unit tests

@ghost
Copy link
Author

ghost commented Nov 17, 2022

Closes #107

@ghost ghost requested a review from outscale-hmi November 18, 2022 10:39
Copy link
Contributor

@outscale-hmi outscale-hmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good for me , just one question about the minimum version of clusterctl ?

Makefile Outdated
@@ -39,8 +39,8 @@ MINIMUM_KUBEBUILDERTOOL_VERSION=1.24.1
MINIMUM_ENVTEST_VERSION=1.23.3
E2E_CONF_FILE_SOURCE ?= ${PWD}/test/e2e/config/outscale-ci.yaml
E2E_CONF_FILE ?= ${PWD}/test/e2e/config/outscale-ci-envsubst.yaml
MINIMUM_CLUSTERCTL_VERSION=1.2.4
MIN_GO_VERSION=1.18.5
MINIMUM_CLUSTERCTL_VERSION=1.1.3
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why this changement ?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No need thanks.

@ghost ghost requested review from outscale-hmi and outscale-mdr November 21, 2022 14:33
@ghost ghost assigned outscale-hmi Nov 24, 2022
Copy link
Contributor

@outscale-hmi outscale-hmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ghost ghost merged commit 3cc42e8 into outscale:main Nov 25, 2022
@ghost ghost added the kind/cleanup Cleanup label Nov 30, 2022
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

Successfully merging this pull request may close these issues.

Use GOPATH in scripts
3 participants