Closed michael-k closed 4 days ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 93.29%. Comparing base (
3724d31
) to head (8106318
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issue #, if available: n/a
Description of changes: The examples no longer worked with cfn-lint v1. The
PropertiesTagsRequired
probably didn't work with v0 either, because it didn't account for resources that use objects as tags, eg. AWS::SSM::Parameter.Instead of
"Tags" in schema.schema["properties"]
I wanted to use(schema.schema.get("tagging") or {}).get("taggable")
, but that doesn't work withAWS::ApiGatewayV2::Stage
("tagging" not in schema.schema
, but the resource supports tags). Happend to have that resource in our test suite, there might be more where this doesn't work. (See also this message at discord.)By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.