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.

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

add "Condition" field to CFN/SAM schemas #360

Open
hauntingEcho opened this issue Mar 25, 2021 · 2 comments
Open

add "Condition" field to CFN/SAM schemas #360

hauntingEcho opened this issue Mar 25, 2021 · 2 comments

Comments

@hauntingEcho
Copy link

the CFN/SAM schemas currently support the Conditions top-level object but not the Condition resource property (the sibling of Type and Properties on any resource).

https://docs.aws.amazon.com/AWSCloudFormation/latest/UserGuide/conditions-section-structure.html

@xrn
Copy link
Contributor

xrn commented Mar 26, 2021

Each resource has property AWSCloudFormationCondition fell free to use it

@hauntingEcho
Copy link
Author

I'm not seeing those in any of the files in the schema folder.

I'm honestly just using the JSON schema files for syntax highlighting in serverless templates - schemastore links those back to here, and is the default JSON/YML schema registry for Jetbrains' IDEs. I'm not sure that a differently-named property would work. From a quick sampling, it does look like each resource's AWSCloudFormationCondition gets marshalled into condition, though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants