Pat Myron

Results 271 comments of Pat Myron

[`cfn-python-lint` added support for `nodejs.12x` here](https://github.com/aws-cloudformation/cfn-python-lint/pull/1204) if that helps https://github.com/martysweet/cfn-lint#depreciationretirement-notice

[`cfn-python-lint` added support for linting multiple templates here](https://github.com/aws-cloudformation/cfn-python-lint/pull/331) if that helps https://github.com/aws-cloudformation/cfn-python-lint/#lint-multiple-files https://github.com/martysweet/cfn-lint#depreciationretirement-notice

> The reason I ask is that I am trying to use `cfn-lint` in a docker container to avoid pip dependency conflicts I'm having Sorry about the delayed response. Not...

relevant code: https://github.com/aws-cloudformation/aws-cfn-lint-visual-studio-code/blob/089b197e2c7df4ec26d35c23ebb77383c724a86e/client/src/extension.ts#L188-L209

Contract test names themselves need to be clearer. Needed to fix my `ListHandler` to get a contract test without `list` in the name to pass, which is not intuitive, even...

> 3. If your Schema contains create only properties, make sure you include a check in the Update Handler for verifying if these properties are not being changed. Why should...

> In terms of checking whether the resource exists, I do think it should be handled by each handler. The error codes and exceptions thrown will be unique for whatever...

Merging/rebasing latest code should get rid of that pylint failure btw. No worries about the contract-integration failures, that's just forks not being able to access the main repo's secrets

places I could find not using [`AWS_DEFAULT_REGION`](https://docs.aws.amazon.com/cli/latest/userguide/cli-configure-envvars.html): https://github.com/aws-cloudformation/cloudformation-cli/blob/49eac51bf8257b0b2cc63cc26c9338eff592d3e1/src/rpdk/core/test.py#L32 https://github.com/aws-cloudformation/cloudformation-cli/blob/49eac51bf8257b0b2cc63cc26c9338eff592d3e1/src/rpdk/core/fragment/generator.py#L115-L116 --- similar to https://github.com/aws-cloudformation/cloudformation-cli/issues/557

> You can check a documentation example [here](https://github.com/aws-cloudformation/aws-cloudformation-resource-providers-cloudformation/tree/master/aws-cloudformation-resourcedefaultversion/docs) where the required property is ignored. Not seeing any differences after this change? [None of the properties look like they're required by...