Skip to content
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

feat: upgrade to go version 1.18 #39

Merged
merged 15 commits into from
Jun 21, 2022
Merged

feat: upgrade to go version 1.18 #39

merged 15 commits into from
Jun 21, 2022

Conversation

scottd018
Copy link
Contributor

The following is addressed with this PR:

Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
cases.Title is set to replace strings.Title, but it does not return the same.  We consider a
'title' of a field of collection.spec.name to read collection.Spec.Name, but the new
method returns collection.Spec.name instead.  Only the first field is capitalized and the
rest are lowercased.  We will use the deprecated method until we can figure out how to
make it return appropriately.

Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
This commit upgrades the kustomize version as the old version is imcompatible with the way it was
importing its packages in go1.18.

It also adjusts the controller-gen options to account for a newer version of controller-gen by
removing the deprecated options.

Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Also centralized the preferred and minimum version variables into a version file
in utils package to assist with future upgrades.

Signed-off-by: Dustin Scott <dustin.scott18@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: upgrade go to 1.18 bug: changing managed labels on component resource does not queue reconciliation
2 participants