Skip to content

Fix CLI startup syntax error#2

Open
MrDiscipline wants to merge 1 commit into
wookiecooking:masterfrom
MrDiscipline:fix-cli-syntax-error
Open

Fix CLI startup syntax error#2
MrDiscipline wants to merge 1 commit into
wookiecooking:masterfrom
MrDiscipline:fix-cli-syntax-error

Conversation

@MrDiscipline

Copy link
Copy Markdown

Summary

  • fix the startup syntax error in bin/lunkcrypt
  • end the first var declaration with a semicolon instead of a comma before the next var

Fixes #1.

Verification

node --check bin/lunkcrypt
node bin/lunkcrypt --help
node bin/lunkcrypt --version

I also packed the branch and installed the tarball into a clean temporary npm project:

npm pack
npm install /path/to/lunkcrypt-1.0.3.tgz
./node_modules/.bin/lunk --help
./node_modules/.bin/lunk --version

Both commands exit 0 after the fix. The currently published lunkcrypt@1.0.3 fails immediately with SyntaxError: Unexpected token 'var'.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CLI fails to start because bin/lunkcrypt has a syntax error

1 participant