Commit edc2761f authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'id-redundant-paranthesis' into 'master'

Fix Style/RedundantParentheses offenses

See merge request gitlab-org/gitlab!36423
parents 611c3a8a 04c39b4e
...@@ -666,12 +666,6 @@ Style/RedundantFreeze: ...@@ -666,12 +666,6 @@ Style/RedundantFreeze:
Style/RedundantInterpolation: Style/RedundantInterpolation:
Enabled: false Enabled: false
# Offense count: 6
# Cop supports --auto-correct.
Style/RedundantParentheses:
Exclude:
- 'ee/app/models/ee/merge_request.rb'
# Offense count: 33 # Offense count: 33
# Cop supports --auto-correct. # Cop supports --auto-correct.
# Configuration parameters: AllowMultipleReturnValues. # Configuration parameters: AllowMultipleReturnValues.
......
...@@ -161,7 +161,7 @@ module EE ...@@ -161,7 +161,7 @@ module EE
end end
def has_dependency_scanning_reports? def has_dependency_scanning_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.dependency_list_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.dependency_list_reports)
end end
def compare_dependency_scanning_reports(current_user) def compare_dependency_scanning_reports(current_user)
...@@ -171,11 +171,11 @@ module EE ...@@ -171,11 +171,11 @@ module EE
end end
def has_license_scanning_reports? def has_license_scanning_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.license_scanning_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.license_scanning_reports)
end end
def has_container_scanning_reports? def has_container_scanning_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.container_scanning_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.container_scanning_reports)
end end
def compare_container_scanning_reports(current_user) def compare_container_scanning_reports(current_user)
...@@ -185,11 +185,11 @@ module EE ...@@ -185,11 +185,11 @@ module EE
end end
def has_sast_reports? def has_sast_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.sast_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.sast_reports)
end end
def has_secret_detection_reports? def has_secret_detection_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.secret_detection_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.secret_detection_reports)
end end
def compare_sast_reports(current_user) def compare_sast_reports(current_user)
...@@ -205,7 +205,7 @@ module EE ...@@ -205,7 +205,7 @@ module EE
end end
def has_dast_reports? def has_dast_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.dast_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.dast_reports)
end end
def compare_dast_reports(current_user) def compare_dast_reports(current_user)
...@@ -221,7 +221,7 @@ module EE ...@@ -221,7 +221,7 @@ module EE
end end
def has_metrics_reports? def has_metrics_reports?
!!(actual_head_pipeline&.has_reports?(::Ci::JobArtifact.metrics_reports)) !!actual_head_pipeline&.has_reports?(::Ci::JobArtifact.metrics_reports)
end end
def compare_metrics_reports def compare_metrics_reports
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment