Remove try... convert of integers (E3002)

In the property Type check, when the property value should be an integer (but it’s not), there is an attempt to convert it:

https://github.com/awslabs/cfn-python-lint/blob/master/src/cfnlint/rules/resources/properties/Properties.py#L56

Since both YAML and JSON just handle these types out the box, the try..except can in theory be removed (since the type’s don’t match, a string is not an integer).

Personally I like the check to be more "strict’, who do other think?

Example (JSON): https://github.com/awslabs/cfn-python-lint/blob/master/test/templates/quickstart/vpc-management.json#L816

Example (YAML): https://github.com/awslabs/cfn-python-lint/blob/master/test/templates/good/resource_properties.yaml#L464

These values can easily be written as actual integers

      - CidrIp:
          Ref: SSHLocation
        FromPort: 22 
        IpProtocol: tcp
        ToPort: 22

[finchd]

Yes, it definitely fails to create the stack: “Value of property Value must be of type String” aws cloudformation validate-template succeeds with no errors either.

[fatbasstard]

Any change this (https://github.com/awslabs/cfn-python-lint/pull/83) is merged before the next release (https://github.com/awslabs/cfn-python-lint/pull/84)?