gitlabhq/gitlabhq

View on GitHub
app/mailers/emails/merge_requests.rb

Summary

Maintainability
A
2 hrs
Test Coverage

Method relabeled_merge_request_email has 5 arguments (exceeds 4 allowed). Consider refactoring.
Open

    def relabeled_merge_request_email(recipient_id, merge_request_id, label_names, updated_by_user_id, reason = nil)
Severity: Minor
Found in app/mailers/emails/merge_requests.rb - About 35 mins to fix

    Method merge_request_status_email has 5 arguments (exceeds 4 allowed). Consider refactoring.
    Open

        def merge_request_status_email(recipient_id, merge_request_id, status, updated_by_user_id, reason = nil)
    Severity: Minor
    Found in app/mailers/emails/merge_requests.rb - About 35 mins to fix

      Method changed_milestone_merge_request_email has 5 arguments (exceeds 4 allowed). Consider refactoring.
      Open

          def changed_milestone_merge_request_email(recipient_id, merge_request_id, milestone, updated_by_user_id, reason = nil)
      Severity: Minor
      Found in app/mailers/emails/merge_requests.rb - About 35 mins to fix

        Method reassigned_merge_request_email has 5 arguments (exceeds 4 allowed). Consider refactoring.
        Open

            def reassigned_merge_request_email(recipient_id, merge_request_id, previous_assignee_ids, updated_by_user_id, reason = nil)
        Severity: Minor
        Found in app/mailers/emails/merge_requests.rb - About 35 mins to fix

          There are no issues that match your filters.

          Category
          Status