search

mapillary / mapillary-python-sdk

A Python 3 library built on the Mapillary API v4 to facilitate retrieving and working with Mapillary data.
MIT License
39 stars 15 forks source link

Simplify PR template #45

Closed gmelodie closed 3 years ago

gmelodie commented 3 years ago

I've been reviewing some PRs today and found their descriptions to be super bloated. It would be easier to just look at the code and see what it does instead (actually, that's what I've been doing -- skipping the PR descriptions altogether). For not, since there's only a handful of people working on this, I see no need for such a template.

In the future, you might want to add a template to help guide the community contributions, but even then I think the template could be much more simple.

Simple is better than complex Complex is better than complicated

Rubix982 commented 3 years ago

Would you suggest no PR template or maybe a better PR template suggestion?

gmelodie commented 3 years ago

I think maybe both, 2 templates: a blank one and a simplified one