Skip to content

Conversation

@sdodsley
Copy link
Member

SUMMARY

Updated dependencies to include Python requirements.
Closes #879

Updated dependencies to include Python requirements.
@sdodsley sdodsley self-assigned this Nov 25, 2025
@sdodsley sdodsley added the Infra label Nov 25, 2025
@sdodsley sdodsley marked this pull request as ready for review November 25, 2025 14:37
@@ -1,8 +1,4 @@
---
version: 1
Copy link

Choose a reason for hiding this comment

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

version is not required for this file (builder does not support different versions of this file yet). See the collection template example here.

Otherwise, the format looks good to me. I did no testing of the changes, however.

@sdodsley
Copy link
Member Author

@Shrews so i am now confused. I tried with version and the Ansible lint test failed...
$ 'version' is a required property. See https://docs.ansible.com/projects/builder/en/latest/definition/

@Shrews
Copy link

Shrews commented Nov 25, 2025

@Shrews so i am now confused. I tried with version and the Ansible lint test failed... $ 'version' is a required property. See https://docs.ansible.com/projects/builder/en/latest/definition/

sigh Ok, just leave it there then. Turns out ansible-lint is not correct in this case. It seems to try to verify the collection meta EE as a full on, singluar EE, when in fact, they are different beasts, though similar structures. It's fine to leave it as if it interrupts your collection development workflow as it will just be ignored by ansible-builder.

@sdodsley
Copy link
Member Author

Thanks for the clarification.

@sdodsley sdodsley merged commit 3e31a11 into master Nov 25, 2025
38 checks passed
@sdodsley sdodsley deleted the ee-changes branch November 25, 2025 17:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect meta/execution-environment.yml file

3 participants