Further to the CI integrations mentioned in #13 (prettier & eslint), this PR makes a few more improvements:
Also check that package-lock.json is unchanged when running NPM install in Node.js v16 (this will fail when packages are installed and this file isn't updated, or when the version in package.json is updated but package-lock.json is not updated to match.
Run TypeScript compilation checks to make sure that everything type-checks.
Run Push & PR checks regardless of branch (I was finding it frustrating trying to determine whether what I'd written was working correctly on my fork until I had done this, and I imagine many others will find the same).
Further to the CI integrations mentioned in #13 (prettier & eslint), this PR makes a few more improvements:
package.jsonis updated butpackage-lock.jsonis not updated to match.Open to discussing any of the above of course :)
Results when invalid TS or
package-lock.jsonare pushed can be found here: https://github.com/s0/nody-greeter/actions/runs/1830680405