-
-
Notifications
You must be signed in to change notification settings - Fork 63
NestJS #12
Comments
@eddiejaoude Do we need Prettier as well in GitHub actions? We can make it run and format the files before every user commits and just add linting and build Action. What do you think? Also, if we have prettier in GitHub Action, we will have to do one additional commit via GitHub Action right? When it format the files? |
@khattakdev I would rather the Action does not change any files 👍 so leave Prettier off the Action. We could always add something like Husky later to run things locally before people commit. |
I agree having automated e2e tests is beneficial, but I'd like to know the reason behind removing unit tests in favor or e2e? |
Ahhh e2e vs unit testing, I have a video coming up about this subject, there is a place for both for sure but there are several considerations to be had |
Yes ideally we should (and can) have both 👍 . Unit testing definitely has the benefit of being able to test boundary conditions within the depths of the code. But I often seen when unit tests pass, but the product still does not work. A good example is https://twitter.com/ThePracticalDev/status/687672086152753152 I think for now we should prioritise e2e tests, as the API is blocking a lot of other projects. |
I think for the e2e testing on the API, we should use
|
I will close this as we have more specific issues for remaining tasks |
api
project from CLIdiscord
with the generate commandresource
docs(remove unit testing)member
without DBtypeorm
docsNotes for the future
The text was updated successfully, but these errors were encountered: