Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Avoid using meta schema on ansible-test integration tests #2156

Merged
merged 2 commits into from
May 18, 2022
Merged

Avoid using meta schema on ansible-test integration tests #2156

merged 2 commits into from
May 18, 2022

Conversation

felixfontein
Copy link
Contributor

Fixes #2155.

It would be even cooler to have a schema for these files, but that can be added later. Usually they only contain dependencies: [list of strings] anyway.

@felixfontein felixfontein requested review from a team as code owners May 17, 2022 20:10
@felixfontein felixfontein mentioned this pull request May 17, 2022
Closed
@ssbarnea ssbarnea added the bug label May 18, 2022
@ssbarnea ssbarnea changed the title Classify meta/main.y(a)ml from ansible-test integration tests as 'test-meta' and not 'meta' Avoid using meta schema on ansible-test integration tests May 18, 2022
@cognifloyd cognifloyd merged commit 679ffc0 into ansible:main May 18, 2022
@felixfontein felixfontein deleted the meta branch May 18, 2022 16:57
@felixfontein
Copy link
Contributor Author

@ssbarnea @cognifloyd thanks for reviewing and merging!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ssbarnea ssbarnea ssbarnea approved these changes

@cognifloyd cognifloyd cognifloyd approved these changes

@cidrblock cidrblock Awaiting requested review from cidrblock cidrblock is a code owner automatically assigned from ansible/devtools

@priyamsahoo priyamsahoo Awaiting requested review from priyamsahoo priyamsahoo is a code owner automatically assigned from ansible/devtools

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

meta schema is wrongly applied to meta/main.yml in ansible-test integration tests
3 participants
@felixfontein @ssbarnea @cognifloyd