Skip to content

Remove unnecessary service requires [passes locally, always fails on CI]

Created by: cirosantilli

and explicit modules MergeRequests::

Rails does magic autoloading based on file paths and we already use it extensively.

I don't understand why but this always passes locally and always fails on CI because of the new edit and remove blob actions.

From the errors it is clear that the file services are loading the wrong base service (base instead of file/base) on the CI because of the wrong number of arguments.

I don't understand why this is so: according to this blog post http://urbanautomaton.com/blog/2013/08/27/rails-autoloading-hell/ Files::BaseService should be searched first.

As it stands it cannot be merged because it would break the CI, but I'd really like to know why it fails only on the CI and not locally...

Merge request reports