Skip to content

Commit

Permalink
FIX: Full installation of cfn-lint package (#429)
Browse files Browse the repository at this point in the history
* FIX: Full installation of cfn-lint package

* FIX: test, npm run all
  • Loading branch information
ScottBrenner authored Aug 19, 2024
1 parent 80dff4e commit c422b7b
Show file tree
Hide file tree
Showing 3 changed files with 7 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -165,7 +165,7 @@ module.exports = {
"pip",
"install",
"--upgrade",
`cfn-lint==${version}`,
`cfn-lint[full]==${version}`,
]);

// Symlink from separate directory so only CFN LINT CLI is added to PATH
Expand Down Expand Up @@ -4332,4 +4332,4 @@ const setup = __nccwpck_require__(7391);

module.exports = __webpack_exports__;
/******/ })()
;
;
2 changes: 1 addition & 1 deletion lib/utils/installCLI.js
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,7 @@ module.exports = {
"pip",
"install",
"--upgrade",
`cfn-lint==${version}`,
`cfn-lint[full]==${version}`,
]);

// Symlink from separate directory so only CFN LINT CLI is added to PATH
Expand Down
5 changes: 4 additions & 1 deletion tests/setup.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -58,7 +58,10 @@ test.each([
expect(io.which).toHaveBeenCalledWith(test.expected.python, true);
expect(exec.exec).toHaveBeenCalledWith(
expect.anything(),
expect.arrayContaining(["install", `cfn-lint==${test.expected.version}`]),
expect.arrayContaining([
"install",
`cfn-lint[full]==${test.expected.version}`,
]),
);
expect(core.addPath).toHaveBeenCalledTimes(1);
});
Expand Down

0 comments on commit c422b7b

Please sign in to comment.