ci(test): add checkJS step to test.yml workflow

[oscard0m]

📝 Summary

• Add checkJS step to test.yml workflow
• Octoherd scripts will be scaffolded with this check in their test.yml file

⚠️ I guess this is blocked by: octoherd/script-setup-renovate#3 (see errors in https://github.com/octoherd/create-octoherd-script/runs/2557871514?check_suite_focus=true)

• Installed @types/node as devDependency

⛱ Motivation and Context

To improve CI in Octoherd's scripts ecosystem to validate and assure JS code merged compiles
Fix #16

[gr2m]

⚠️ I guess this is blocked by: octoherd/script-setup-renovate#3 (see errors in https://github.com/octoherd/create-octoherd-script/runs/2557871514?check_suite_focus=true)

I think these are two different things. The code in this repository is node only, it's not an @octoherd script. In order to make the TS check pass, we can add the missing types as dev dependencies to this repository

[oscard0m]

⚠️ I guess this is blocked by: octoherd/script-setup-renovate#3 (see errors in https://github.com/octoherd/create-octoherd-script/runs/2557871514?check_suite_focus=true)

I think these are two different things. The code in this repository is node only, it's not an @octoherd script. In order to make the TS check pass, we can add the missing types as dev dependencies to this repository

You are right. Types added 🔁 👓

[gr2m]

👍🏼

[github-actions]

🎉 This PR is included in version 1.4.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀