fix: use isinstance check instead of type

[jlhood]

Issue #, if available: #1410

Note, had to run make black to correct formatting errors elsewhere in the project in order for make pr to pass.

Checklist:

• [n/a] Write/update tests
• [ x ] make pr passes
• [n/a] Update documentation
• [n/a] Verify transformed template deploys and application functions as expected
• [n/a] Add/update example to examples/2016-10-31

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[keetonian]

Closed & reopened PR to kick off new travis run.

[codecov-io]

Codecov Report

Merging #1411 into develop will decrease coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop    #1411      +/-   ##
===========================================
- Coverage    94.42%   94.42%   -0.01%     
===========================================
  Files           78       78              
  Lines         4664     4663       -1     
  Branches       930      930              
===========================================
- Hits          4404     4403       -1     
  Misses         121      121              
  Partials       139      139

Impacted Files	Coverage Δ
samtranslator/translator/translator.py	98.38% <100%> (-0.02%)	⬇️
samtranslator/translator/logical_id_generator.py	90.9% <100%> (ø)	⬆️
samtranslator/model/sam_resources.py	94.06% <100%> (ø)	⬆️
samtranslator/__init__.py	100% <100%> (ø)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update dd5f016...7a7adb3. Read the comment docs.

[kddejong]

aws-cloudformation/cfn-lint#1325

[keetonian]

@kddejong thanks for bringing this up. I will dig into this.

Edit: it looks like this PR fixes the issue you reference. We will see if we can get this in soon.

[kddejong]

Thanks @keetonian and @shwetaskatdare