mrman511 / event-manager-ruby

0 stars 0 forks source link

12 test todo model and controller #13

Closed mrman511 closed 3 weeks ago

fulrich commented 4 weeks ago

Generally you will want to include tests in the PR that implements the behaviour. In general I wouldn't approve PRs that don't include tests for the code being added.

If you want to scale down the size of your PRs (a great thing to do) you can cut back on the features being added in that PR. For example: Instead of a create and fetch API with no tests just create a PR with create and tests then follow up with a PR that includes fetch and tests.