Skip to content

feat: CFN Lint Support - Revert of previous revert - #4556

Merged
sriram-mv merged 7 commits into
aws:developfrom
sriram-mv:cfn_lint_revert_pr
Jan 10, 2023
Merged

feat: CFN Lint Support - Revert of previous revert#4556
sriram-mv merged 7 commits into
aws:developfrom
sriram-mv:cfn_lint_revert_pr

Conversation

@sriram-mv

Copy link
Copy Markdown
Contributor

Which issue(s) does this change fix?

Reverts #4494

Why is this change necessary?

Preparation of cfn-lint support

How does it address the issue?

Addresses conflicts and installer issues along with the revert of the actual previous PR

What side effects does this change have?

N/A

Mandatory Checklist

PRs will only be reviewed after checklist is complete

  • Add input/output type hints to new functions/methods
  • Write design document if needed (Do I need to write a design document?)
  • Write/update unit tests
  • Write/update integration tests
  • Write/update functional tests if needed
  • make pr passes
  • make update-reproducible-reqs if dependencies were changed
  • Write documentation

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

@sriram-mv
sriram-mv requested a review from a team as a code owner January 10, 2023 17:40
@github-actions github-actions Bot added pr/internal area/validate sam validate command labels Jan 10, 2023
@mndeveci mndeveci changed the title CFN Lint Support - Revert of previous revert. feat: CFN Lint Support - Revert of previous revert Jan 10, 2023

@qingchm qingchm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

------

** libBZ2; version 1.1.0 -- https://gitlab.com/bzip2/bzip2
Copyright c 1996-2007 Julian Seward
 
--------------------------------------------------------------------------

This program, "bzip2", the associated library "libbzip2", and all
documentation, are copyright (C) 1996-2010 Julian R Seward.  All
rights reserved.

Redistribution and use in source and binary forms, with or without
modification, are permitted provided that the following conditions
are met:

1. Redistributions of source code must retain the above copyright
   notice, this list of conditions and the following disclaimer.

2. The origin of this software must not be misrepresented; you must
   not claim that you wrote the original software.  If you use this
   software in a product, an acknowledgment in the product
   documentation would be appreciated but is not required.

3. Altered source versions must be plainly marked as such, and must
   not be misrepresented as being the original software.

4. The name of the author may not be used to endorse or promote
   products derived from this software without specific prior written
   permission.

THIS SOFTWARE IS PROVIDED BY THE AUTHOR ``AS IS'' AND ANY EXPRESS
OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE IMPLIED
WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE
ARE DISCLAIMED.  IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR ANY
DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE
GOODS OR SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS
INTERRUPTION) HOWEVER CAUSED AND ON ANY THEORY OF LIABILITY,
WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT (INCLUDING
NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.

Julian Seward, jseward@acm.org
bzip2/libbzip2 version 1.1.0 of 6 September 2010

--------------------------------------------------------------------------

Let's add bzip2 into our third party license file as well

@sriram-mv
sriram-mv enabled auto-merge (squash) January 10, 2023 21:24
@sriram-mv
sriram-mv merged commit 376c394 into aws:develop Jan 10, 2023
Comment thread requirements/base.txt
tzlocal==3.0

#Adding cfn-lint dependency for SAM validate
cfn-lint==0.72.2

@jplock jplock Jan 12, 2023

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

does it make sense to pin the patch version considering how frequently cfn-lint gets updated due to automated CloudFormation spec updates?

aws-sam-cli 1.69.0 requires cfn-lint==0.72.2, but you have cfn-lint 0.72.8 which is incompatible.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/validate sam validate command pr/internal

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants