Remove inheritance chain from SquashService
What does this MR do and why?
Remove inheritance chain from SquashService
MergeRequests::SquashService
was inheriting from MergeRequests::BaseService
and by that it had a lot of methods and responsibilities that it didn't need.
The refactor allowed to also improve the interface of the service object taking only the data it really needs explicitly.
MR acceptance checklist
This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.
-
I have evaluated the MR acceptance checklist for this MR.