Skip to content

Commit

Permalink
Merge pull request #4 from andybelltree/filter-changed-files
Browse files Browse the repository at this point in the history
Fix boolean parameters being passed in as strings
  • Loading branch information
andybelltree authored Nov 21, 2021
2 parents 87352ea + 6d980f3 commit 9bef174
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 4 deletions.
4 changes: 2 additions & 2 deletions dist/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -23106,8 +23106,8 @@ async function main$1() {
const githubClient = new github_2(token);
const lcovFile = core$1.getInput("lcov-file") || "./coverage/lcov.info";
const baseFile = core$1.getInput("lcov-base");
const shouldFilterChangedFiles = core$1.getInput("filter-changed-files");
const shouldDeleteOldComments = core$1.getInput("delete-old-comments");
const shouldFilterChangedFiles = core$1.getInput("filter-changed-files").toLowerCase() === 'true';
const shouldDeleteOldComments = core$1.getInput("delete-old-comments").toLowerCase() === 'true';
const title = core$1.getInput("title");
const maxUncoveredLines = core$1.getInput("max-uncovered-lines");
if (maxUncoveredLines && isNaN(parseInt(maxUncoveredLines))) {
Expand Down
6 changes: 4 additions & 2 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,8 +15,10 @@ async function main() {
const githubClient = new GitHub(token)
const lcovFile = core.getInput("lcov-file") || "./coverage/lcov.info"
const baseFile = core.getInput("lcov-base")
const shouldFilterChangedFiles = core.getInput("filter-changed-files")
const shouldDeleteOldComments = core.getInput("delete-old-comments")
const shouldFilterChangedFiles =
core.getInput("filter-changed-files").toLowerCase() === "true"
const shouldDeleteOldComments =
core.getInput("delete-old-comments").toLowerCase() === "true"
const title = core.getInput("title")
const maxUncoveredLines = core.getInput("max-uncovered-lines")
if (maxUncoveredLines && isNaN(parseInt(maxUncoveredLines))) {
Expand Down

0 comments on commit 9bef174

Please sign in to comment.