Skip to content

Naive single quotes - #52

Merged
theskumar merged 3 commits into
theskumar:masterfrom
Flimm:naive-single-quotes
Mar 30, 2017
Merged

Naive single quotes#52
theskumar merged 3 commits into
theskumar:masterfrom
Flimm:naive-single-quotes

Conversation

@Flimm

@Flimm Flimm commented Mar 30, 2017 •
edited
Loading

Copy link
Copy Markdown
Contributor

This fixes #35

Note, that this is still far from an SH-compatible parser, but this at least fixes this particular issue.

To be an SH-compatible parser, these kinds of things would have to be fixed:

  • HELLO=WORLD shoud be treated differently from HELLO= WORLD
  • HELLO=${WORLD} should be treated differently from HELLO='${WORLD}'
  • The parsing of a POSIX variable should take a previous POSIX variable in precedence over an environment variable.
  • And so on.

@coveralls

coveralls commented Mar 30, 2017 •
edited
Loading

Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 82.418% when pulling a2096aa on Flimm:naive-single-quotes into 9552db8 on theskumar:master.

@theskumar

theskumar commented Mar 30, 2017

Copy link
Copy Markdown