-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Remove cfn lint #1034
Remove cfn lint #1034
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #1034 +/- ##
========================================
Coverage 94.82% 94.82%
========================================
Files 69 69
Lines 3244 3244
Branches 632 632
========================================
Hits 3076 3076
Misses 87 87
Partials 81 81 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Are we going to open an issue to find a way to add cfn-lint back? It's very helpful to lint output templates to make sure they're correct... We could install it separately as part of the Travis run and then just run it against all output templates. That way it's not part of our dependency closure. Is that a difficult change? Could we just do that as part of this PR instead of only removing it?
cfn-lint is no longer a test dependency since [1], and the test suite is green now. [1] aws/serverless-application-model#1034
python-cfn-lint is no longer needed [1]. It might be re-added in the future, though [2]. [1] aws/serverless-application-model#1034 [2] aws/serverless-application-model#1042 git-svn-id: file:///srv/repos/svn-community/svn@505539 9fca08f4-af9d-4005-b8df-a31f2cc04f65
Issue #, if available:
#1008
Description of changes:
Removes cfn-lint test dependency
Description of how you validated changes:
Tests run and pass
Checklist:
make pr
passesBy submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.