Skip to content

Commit

Permalink
add comment about browser and quotes
Browse files Browse the repository at this point in the history
  • Loading branch information
bahmutov committed Nov 16, 2019
1 parent 3732300 commit f3810f6
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions dist/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -1853,6 +1853,8 @@ const runTests = () => {
const browser = core.getInput('browser')
if (browser) {
cmd.push('--browser')
// TODO should browser be quoted?
// If it is a path, it might have spaces
cmd.push(browser)
}

Expand Down
2 changes: 2 additions & 0 deletions index.js
Original file line number Diff line number Diff line change
Expand Up @@ -250,6 +250,8 @@ const runTests = () => {
const browser = core.getInput('browser')
if (browser) {
cmd.push('--browser')
// TODO should browser be quoted?
// If it is a path, it might have spaces
cmd.push(browser)
}

Expand Down

0 comments on commit f3810f6

Please sign in to comment.