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

chores: Update build submodule and up up cli version #2067

Merged

Conversation

ChristianAvila
Copy link
Contributor

@ChristianAvila ChristianAvila commented Jun 9, 2024

Description of your changes

  • Update build module to use the latest supported version and change the up version that is compatible with docker > 1.25
  • vendor and vendor.check were deprecated, updated the CI to use the new version modules.download and modules.check
Go Targets:
    modules.download Download Go modules.
    modules.check    Fail the build if Go modules have changed.

I have:

  • Read and followed Crossplane's contribution process.
  • Run make reviewable test to ensure this PR is ready for review.

How has this code been tested

  • Build in MacOS with Docker
  • Run actions
image

@ChristianAvila ChristianAvila force-pushed the update-build-versions branch 3 times, most recently from df9c2f7 to ae39061 Compare June 13, 2024 23:32
Copy link
Collaborator

@MisterMX MisterMX left a comment

Choose a reason for hiding this comment

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

LGTM. Thank you very much @ChristianAvila!

up cli version is now compatible with docker >= 1.25

Signed-off-by: ChristianAvila <[email protected]>
make vendor was deprecated in favor of modules.download and vendor.check
to modules.check

Signed-off-by: ChristianAvila <[email protected]>
@ChristianAvila
Copy link
Contributor Author

Thanks @MisterMX, I noticed an error in the check diff and e2e-tests. Both have been fixed in my latest commit. Can you please help me with a review?

Both actions are now running smoothly in my repository.

@MisterMX MisterMX merged commit 5eb25dc into crossplane-contrib:master Jul 24, 2024
9 checks passed
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.

2 participants