Showing results for 
Search instead for 
Did you mean: 
Sign up Log in

Files excluded via new "Exclude files" shouldn't count towards "this is a large pull request" limit

I'm New Here
I'm New Here
Those new to the Atlassian Community have posted less than three times. Give them a warm welcome!
June 23, 2021

I am currently working with a PR that has 309 changed files, 247 of which are tracking metadata in an ignored directory (specifically, it is the `.todo-lint` directory for `ember-template-lint`).

This, of course, flips it into a "large pull request", even though 80% of the file diffs are totally useless to a human reviewer. I was excited to find out about the Exclude Files functionality, and excluded that directory.

Unfortunately, the implementation of the Exclude Files functionality makes it totally useless for my use case. It simply hides the diff, as if it were a too-large diff, and the "excluded" files still show up in the list of diffs and result in the PR being "too large"

Without the excluded files, it would be a reasonably-sized diff that would be perfectly suitable for the normal pull request view.

I think the Exclude Files is a great feature, but I think its implementation hampers its usefulness. It would be much more useful if I was given the usual (non-large) pull request experience with a note that said "247 files have been excluded, click here to see them" or something, instead of being forced to review the 247 ignored files one by one, in addition to the rest of the diffs that I actually care about.

I can see how this features is useful for certain cases (concatenation/minification?) where the number of ignored files is small, but I imagine there are plenty of use cases where that is not the case. If excluded files were excluded from the pull request size for the purposes of determining if it is a "large" pull request or not, I think it would be a great improvement, and certainly would be for my use case.




Log in or Sign up to comment
AUG Leaders

Atlassian Community Events