• Markus Koller's avatar
    Drop app/models/integrations from eager load paths · d4b77f8f
    Markus Koller authored
    The models in this folder all use the new `Integrations::` namespace,
    so they can be properly auto-loaded without having to add the path to
    `eager_load_paths`. This was only needed for the old `project_services`
    directory, which didn't use namespaces.
    
    This fixes errors like this when referencing integration classes in the
    toplevel scope:
    
    ```
    LoadError: Unable to autoload constant Github, expected ee/app/models/integrations/github.rb to define it
    ```
    
    Instead we'll now get:
    
    ```
    NameError: uninitialized constant Github
    ```
    
    The previous workaround for the Jira integration to create an empty
    module in `app/services/jira.rb` isn't needed anymore, referencing a
    service like `Jira::requests::Projects::ListService` still works without
    raising an error.
    d4b77f8f
application.rb 18.2 KB