Skip to content

Conversation

tonyredondo
Copy link
Member

@tonyredondo tonyredondo commented Feb 18, 2025

What does this PR do?

Adds a check for the required go version for orchestrion to bail out if the requirements are not met

Adds a check for the required go version for orchestrion

Motivation

Currently we always try to install orchestrion to the target Go project, but orchestrion (following the Go's support guidelines) only supports the latest stable version and the previous one at the moment of the orchestrion release. This mean that if we try to install orchestrion in an old Go project with an old Go version we currently crashes the build.

This PR tries to avoid that by checking the required go version before the orchestrion installation and bailing out safely without breaking the test job.

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

@tonyredondo tonyredondo self-assigned this Feb 18, 2025
@tonyredondo tonyredondo requested a review from a team as a code owner February 18, 2025 15:01
@tonyredondo tonyredondo merged commit fa79955 into main Feb 18, 2025
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