search

bitovi / querystring-parser

MIT License
8 stars 5 forks source link

Set up CI #1

Closed phillipskevin closed 2 years ago

phillipskevin commented 2 years ago

When you make a PR, it should run linting and unit tests based on a GitHub action

phillipskevin commented 2 years ago

A couple improvements I think we should make:

  1. run the lint/test after PRs are merged to main as well as what is happening now
  2. rename the CI Workflow to Tests so the badge is a little cleaner
  3. add a badge to the README
  4. use the ?branch=main param on the badge (this won't work without number 1 above), but right now if you put in a PR with breaking tests, the badge will show CI as failing