Skip to content

Remove inheritance chain from SquashService

Fabio Pitino requested to merge refactor-mr-squash-service into master

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.

Merge request reports