@cclauss do you rember why you add it?
538969a#diff-50c86b7ed8ac2cf95bd48334961bf0530cdc77b5a56f852c5c61b89d735fd711

#520 (comment)

I think it was an indirect dependency.

Hi again,
Removing transitive dependencies from configurations files to let pip manage them automatically, helps keeping the dependency list clean and avoiding version conflicts between packages.

I definitely remember situations where dependencies had version ranges which ended up allowing some indirect dependency which then caused things to break - so while prospector does not use it directly there might have been some strange set of conditions where this constraint was necessary despite it not being used directly.

That's my vague guess at the reasons it was there, but I'm all for removing this.

I failed to properly review the MR that introduced this, I should have asked or clarified the reason to add this dependency at the time. Gitpython is an optional dependency for pylint primer tests, that might be the reason it was added but multiple years later the historical context that might have been obvious at the time (another pr jobs failing perhaps ?) is lost and impliciteness is not enough anymore.