Skip to content
This repository has been archived by the owner on Dec 2, 2022. It is now read-only.

tags can be either string or list of strings #236

Merged
merged 4 commits into from
Apr 13, 2022
Merged

Conversation

MarkusTeufelberger
Copy link
Contributor

I suspect this was an oversight?

There are also "array-only" tags definitions in ansible-meta, ansible-lint and ansible-galaxy. These might also be "string OR array-of-strings" type, but I didn't check or verify those.

@MarkusTeufelberger MarkusTeufelberger requested review from ssbarnea and a team as code owners April 13, 2022 11:44
@ssbarnea ssbarnea added the bug Something isn't working label Apr 13, 2022
@ssbarnea
Copy link
Member

That file is pending deprecation as the new one already got the fix, but we did not switch to it yet.

I also added some tests to your change.

@ssbarnea ssbarnea changed the title "tags" can be a string too when listing roles tags can be either string or list of strings Apr 13, 2022
@MarkusTeufelberger
Copy link
Contributor Author

https://github.com/ansible/schemas/issues/207#issuecomment-1095216114 sounded like this might still be on hold.

schemas/f/ansible.json

Lines 590 to 596 in 3da010f

"tags": {
"items": {
"type": "string"
},
"title": "Tags",
"type": "array"
},
looks like this isn't fixed there either? I'll look into adding/extending test cases...

Markus Teufelberger added 3 commits April 13, 2022 16:48
Signed-off-by: Markus Teufelberger <[email protected]>
Signed-off-by: Markus Teufelberger <[email protected]>
Signed-off-by: Markus Teufelberger <[email protected]>
@MarkusTeufelberger
Copy link
Contributor Author

Alright, should be done + tested now, Ready for squashing and merging.

@ssbarnea ssbarnea merged commit 95d382a into ansible:main Apr 13, 2022
@ssbarnea ssbarnea mentioned this pull request Apr 13, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants