search

mockito / mockito-kotlin

Using Mockito with Kotlin
MIT License
3.09k stars 198 forks source link

Update PR template #504

Closed TWiStErRob closed 7 months ago

TWiStErRob commented 7 months ago

Is this line still relevant? When this line was added the project was using normal merges for PRs, now that they're being squashed to main, is there a reason to keep this line? (I was spending time making the commits really nice because I thought they'll be normal merged.)

Also, I recommend changing to this setting (regardless of this PR getting merged or not): image because the others bring in noise to the commits message and the default has inconsistent behavior between 1 commit and many commits. It might be set already.

TimvdLippe commented 7 months ago

Indeed irrelevant, let's remove it