Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Fix list of non-strings in args rule #4279

Merged
merged 1 commit into from
Aug 1, 2024
Merged

Fix list of non-strings in args rule #4279

merged 1 commit into from
Aug 1, 2024

Conversation

johnbyrneio
Copy link
Contributor

Fixes #4229 where lists of non-strings passed to an argument as a variable will fail.

@johnbyrneio johnbyrneio requested a review from a team as a code owner July 31, 2024 05:10
@johnbyrneio johnbyrneio requested review from audgirka and priyamsahoo and removed request for a team July 31, 2024 05:10
@github-actions github-actions bot added the bug label Jul 31, 2024
@johnbyrneio johnbyrneio mentioned this pull request Jul 31, 2024
Closed
@alisonlhart alisonlhart force-pushed the fix-jinja-list-element-skip branch from f5f6325 to de455d0 Compare August 1, 2024 12:46
@sonarqubecloud SonarQubeCloud
Copy link

sonarqubecloud bot commented Aug 1, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

@alisonlhart alisonlhart merged commit 38c31e0 into ansible:main Aug 1, 2024
26 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@alisonlhart alisonlhart alisonlhart approved these changes

@audgirka audgirka Awaiting requested review from audgirka audgirka 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
🧰 devtools project board
Archived in project
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Misreports a list of ints variable as <class 'str'>
2 participants
@johnbyrneio @alisonlhart