src-d / ghsync

GitHub API v3 > PostgreSQL
https://sourced.tech
GNU General Public License v3.0
9 stars 8 forks source link

Use per-page=100 for all list endpoints #19

Closed smacker closed 5 years ago

smacker commented 5 years ago

It's possible to add some abstraction but it will introduce unnecessary complexity. So I just changed options for all list endpoints.

Fix: #3

Signed-off-by: Maxim Sukharev max@smacker.ru

carlosms commented 5 years ago

Maybe not a complex abstraction, but what about having one constant in some common go file?

smacker commented 5 years ago

tbh I don't see much value in it but changed as you proposed.