Feature/gherkin linting - Githubissues

wp-cli / scaffold-package-command

Scaffolds WP-CLI commands with functional tests, full README.md, and more.

MIT License

68 stars 19 forks source link

Feature/gherkin linting #189

Closed Lewiscowles1986 closed 3 years ago

Lewiscowles1986 commented 3 years ago

Proposal

Fixes #82

Uses GitHub action (can be merged into single file. Kept separate so potential merge conflicts can be avoided
Uses NPM / JS (can pivot and search for something else if something suitable exists)
Adds config file (which may be in need of iteration, it was borrowed from Moodle, but had some spelling mistakes I fixed to avoid errors in a likely outdated file)

Let me know if this is unsuitable. If you really want to have fun and test this:

clone the branch
npm install
npm run lint && echo success || echo failure (should contain "success" and not contain "failure")
change the indentation of a .feature file (I tested by removing all indentation)
npm run lint && echo success || echo failure (should contain "failure")

schlessera commented 3 years ago

Thanks for the PR, @Lewiscowles1986 !