I think we should update the PR Template to include a checklist of things that should be checked before we approve PR?
E.g.
[ ] Spelling Punctuation and Grammar (SPG) / no typos
[ ] PEP8 Style Guide for Python Code
[ ] Tested on Quadcopter
Mainly to stop issues like #193
I think there are some automated checkers you can get which can also block PR's from being merged, but these might be OTT?
I seen this but not sure how to implement it and again it might be OTT.
Describe the solution you'd like
I think we should update the PR Template to include a checklist of things that should be checked before we approve PR?
E.g.
Mainly to stop issues like #193
I think there are some automated checkers you can get which can also block PR's from being merged, but these might be OTT? I seen this but not sure how to implement it and again it might be OTT.
What's your thoughts?