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

Make Renovate play nice with Earthly #5754

Merged
merged 2 commits into from
May 29, 2024

Conversation

negz
Copy link
Member

@negz negz commented May 29, 2024

Description of your changes

This PR contains two fixes.

First, it runs earthly rather than make after bumping Go or golangci-lint. Second, it (hopefully) fixes the version matching logic for the versioned tools in Earthfile.

I have:

  • Read and followed Crossplane's contribution process.
  • Run earthly +reviewable to ensure this PR is ready for review.
  • Added or updated unit tests.
  • Added or updated e2e tests.
  • Linked a PR or a docs tracking issue to document this change.
  • Added backport release-x.y labels to auto-backport this PR.

Need help with this checklist? See the cheat sheet.

Signed-off-by: Nic Cope <nicc@rk0n.org>
@negz negz requested a review from a team as a code owner May 29, 2024 03:47
@negz negz requested a review from turkenh May 29, 2024 03:47
@negz
Copy link
Member Author

negz commented May 29, 2024

@phisco @enesonus could you sanity check this for me. 🙂 I noticed e.g. #5752 is wrong.

@negz negz changed the title Fix Earthfile Renovate version matching and extraction Make Renovate play nice with Earthly May 29, 2024
Signed-off-by: Nic Cope <nicc@rk0n.org>
@negz negz force-pushed the third-rock-from-the-sun branch from a309234 to 1cb5caa Compare May 29, 2024 04:41
Copy link
Contributor

@phisco phisco left a comment

Choose a reason for hiding this comment

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

looks fine, let's see if it also works 😆 🤞 🙏

@enesonus
Copy link
Contributor

It looks fine to me also 👍

@negz negz merged commit dc9c523 into crossplane:master May 29, 2024
19 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.

3 participants