search

csstools / postcss-tape

Quickly test PostCSS plugins
Creative Commons Zero v1.0 Universal
15 stars 4 forks source link

Fix CLI typo in README.md #4

Closed wesleyboar-fka-iosulfur closed 5 years ago

wesleyboar-fka-iosulfur commented 5 years ago

The CLI (i.e. "arguments") instructions included an = sign; the equals sign is not accurate nor necessary. Just to be sure, I tested:

  1. I copied .tape.js as .tape.test.js, and removed several test cases.
  2. Command postcss-tape --config=./tape.test.js used existing /.tape.js (saw many test cases).
  3. Command postcss-tape --config ./tape.test.js used new file /.tape.test.js (saw few test cases).
wesleyboar-fka-iosulfur commented 5 years ago

Well, I am surprised. I did not expect a README.md change to fail a build. I think (hope) there was a commit, before my PR, that did not get built, but breaks the build process.

I looked into build log and Git history. From commits to master, I found commit 595171b, which looks suspicious, but I am not keen (nor awake) enough to be sure if it is the culprit.