Skip to content

Conversation

@cerealsnow
Copy link
Contributor

@cerealsnow cerealsnow commented Aug 13, 2025

Description of your changes

References #22

Synced repository with upjet-provider-template@90fdc76
Updated upjet to v1.9.0

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

  • e2e tests using uptest

@cerealsnow
Copy link
Contributor Author

The golangci configuration in the upjet-provider-template was severely outdated. I bumped the golangci version to v2 and refactored the general configuration. I used this commit from provider-upjet-github as a reference, since they had already migrated to golangci v2.

The make reviewable test command should now work as expected. As part of the migration, I also made slight formatting changes to some files in the config/ folder, because the formatter goimports enforces grouping of imports into external and local modules.

@codecov-commenter
Copy link

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@anton-sidelnikov anton-sidelnikov added the gate zuul merger label Aug 13, 2025
@anton-sidelnikov anton-sidelnikov merged commit 3e15897 into opentelekomcloud:main Aug 13, 2025
12 of 13 checks passed
@cerealsnow cerealsnow deleted the chore/upjet-provider-template@90fdc76 branch August 29, 2025 06:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

gate zuul merger

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants