Skip to content

Conversation

@pacostas
Copy link
Member

@pacostas pacostas commented Oct 1, 2025

Summary

This PR handles differently the the dependencies based on the apiVersion if its greater than 10 or not. Specifically:

  • If api > 10 it filters the dependencies based on the target metadata values and in case on the dependencies the os and arch is not specified, then does not do a strict checking and as a result the dependency will get accepted
  • If api < 10 it filters the dependencies based on stacks and based on the platform. As a result if the stack does not match it will check if the platform matches, and due to we do not also do a strict check, if the dependencies do not include os and arch, the dependency will get accepted.

Use Cases

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
  • I have added an integration test, if necessary.
  • I have reviewed the styleguide for guidance on my code quality.
  • I'm happy with the commit history on this PR (I have rebased/squashed as needed).

@pacostas pacostas requested a review from a team as a code owner October 1, 2025 16:24
@pacostas pacostas requested review from jericop and modulo11 October 1, 2025 16:24
@pacostas pacostas force-pushed the error-loggin-targets branch from cc95fbf to d816724 Compare October 1, 2025 16:24
@pacostas pacostas force-pushed the error-loggin-targets branch from d816724 to 7e4c44e Compare October 1, 2025 16:24
@pacostas pacostas added the semver:patch A change requiring a patch version bump label Oct 1, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

semver:patch A change requiring a patch version bump

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant