search

aws-cloudformation / cfn-lint

CloudFormation Linter
MIT No Attribution
2.43k stars 588 forks source link

Default fn validator context will be not strict type checking #3386

Closed kddejong closed 3 months ago

kddejong commented 3 months ago

Issue #, if available: fix #3383

Description of changes:

Sub has strict types because you can't always use a number or boolean value. When nesting other functions in the Sub that strict type validation should be back to False. This will make sure the default to functions items is false and will still allow Fn Sub to be strict as needed.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

codecov[bot] commented 3 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Please upload report for BASE (main@3dfb0e4). Learn more about missing BASE report.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3386 +/- ## ======================================= Coverage ? 93.24% ======================================= Files ? 323 Lines ? 10992 Branches ? 2295 ======================================= Hits ? 10249 Misses ? 415 Partials ? 328 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.