We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
We should validate the example IaC:
sam validate --lint
sam build
Not only for SAM but for others too.
Make sure the provided templates (IaC) are working.
Governance, Tests
Use sam validate --lint (see https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/validate-cfn-lint.html and https://github.com/aws-cloudformation/cfn-lint#config-file) + sam build for SAM.
See also https://awslabs.github.io/serverless-rules/cfn-lint/
For terraform, use terraform validate and tflint.
terraform validate
tflint
The text was updated successfully, but these errors were encountered:
Please assign this to me.
Sorry, something went wrong.
note that it is done for terraform already: https://github.com/aws-powertools/powertools-lambda-java/blob/main/.github/workflows/pr_build.yml#L82-L109
Also, if you could extract this plus what you will do for SAM in a pr_iac_lint.yml, so that we don't have all these if java 11 everywhere.
pr_iac_lint.yml
if java 11
jasoniharris
No branches or pull requests
Summary
We should validate the example IaC:
sam validate --lint
sam build
(we already have the compilation with maven/gradle)Not only for SAM but for others too.
Why is this needed?
Make sure the provided templates (IaC) are working.
Which area does this relate to?
Governance, Tests
Solution
Use
sam validate --lint
(see https://docs.aws.amazon.com/serverless-application-model/latest/developerguide/validate-cfn-lint.html and https://github.com/aws-cloudformation/cfn-lint#config-file) +sam build
for SAM.See also https://awslabs.github.io/serverless-rules/cfn-lint/
For terraform, use
terraform validate
andtflint
.Acknowledgment
The text was updated successfully, but these errors were encountered: