ansible-community / antsibull-build

Tooling for building various things related to ansible
GNU General Public License v3.0
60 stars 31 forks source link

Add collection-meta.yaml linter #617

Closed felixfontein closed 1 month ago

felixfontein commented 1 month ago

Ensures that collection-meta.yaml has the right order, the right structure, and that all kind of conditions are right.

Assumes the format from https://github.com/ansible-community/ansible-build-data/pull/450. That PR also fixes the order.

Note that this makes pydantic 2+ a required dependency, and thus also antsibull-core 3+.

gotmax23 commented 1 month ago

Thanks for addressing the feedback. I'll try to take a look later today or tomorrow now that I'm back from Labor Day travel.

felixfontein commented 1 month ago

Failing CI is caused by https://github.com/CiscoDevNet/intersight-ansible/issues/137.

gotmax23 commented 1 month ago

Thanks for working on this and for applying all my feedback

felixfontein commented 1 month ago

@gotmax23 thanks for reviewing this!