Closed fishmonger45 closed 2 years ago
deploy action has been split to a separate file; no idea if it works or not :cowboy_hat_face:
Please review. I am aware that there are some extensions that are required to be made to the test suite due to new endpoints being introduced. These will be done in a separate PR because we really need some base level tests in the project.
repinging jchad because i suspect it didn't work out of the box because you are on windows? mb. have added crossenv now.
Yeah I am currently on windows 😢. It works well with cross-env, I'm just investigating the issues with the tests that are being skipped
Description
npm run coverage
for a local coverage report. I don't feel the need to put this in CI/CD because I'm probably the only one who's going to be using it.validate.test.ts
should be removed when we have proper types in backendFixes/resolves #97
Type of change
Please delete options that are not relevant.
Checklist:
Leave blank if not applicable
I have completed these steps when making this pull request:
Before opening the PR for review: