Skip to content
This repository has been archived by the owner on Aug 27, 2024. It is now read-only.

Feature request: check how much lines of code were changed. #530

Open
drummerwolli opened this issue Mar 14, 2018 · 3 comments · May be fixed by #570
Open

Feature request: check how much lines of code were changed. #530

drummerwolli opened this issue Mar 14, 2018 · 3 comments · May be fixed by #570

Comments

@drummerwolli
Copy link
Member

In our team, we have sometimes the problem that we have Pull Requests which are way too big. To enforce the developers to commit smaller chunks for review, it would be interesting to check the size of the PR.

It is possible to get the additions and deletions in a PR via the GH API: https://developer.github.com/v3/pulls/#get-a-single-pull-request

Would it be possible to include a check where i can set the maximum (maybe for someone else also minimum) number of lines changed / additions and deletions?

@drummerwolli
Copy link
Member Author

also restricting changed_files(same call) could be an additional option

@sathish-kumar-subramani
Copy link

sathish-kumar-subramani commented Oct 22, 2018

Hi @lotharschulz,

I can take up this issue.
Any pointers should I know before starting?

So, another method in PullRequestTasks to get additions, deletions and changed_files from pull-request payload and verify them to be less than config.pull-request.tasks.maximum-additions, maximum-deletions and maximum-changed-files should do the trick, right?

@drummerwolli drummerwolli linked a pull request Feb 23, 2019 that will close this issue
@lotharschulz
Copy link
Contributor

@sathish-kumar-subramani I ask you to align with the code maintainers https://github.com/zalando/zappr/blob/master/MAINTAINERS with regards to the pointers you requested. Sorry for answering that late.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants