Commit 2c997121 authored by Peter Leitzen's avatar Peter Leitzen

Merge branch 'rails-logger-cop-11' into 'master'

Use applogger in groups destroy service and label create service

See merge request gitlab-org/gitlab!32195
parents c581e4a4 88b597f6
...@@ -6,7 +6,7 @@ module Groups ...@@ -6,7 +6,7 @@ module Groups
def async_execute def async_execute
job_id = GroupDestroyWorker.perform_async(group.id, current_user.id) job_id = GroupDestroyWorker.perform_async(group.id, current_user.id)
Rails.logger.info("User #{current_user.id} scheduled a deletion of group ID #{group.id} with job ID #{job_id}") # rubocop:disable Gitlab/RailsLogger Gitlab::AppLogger.info("User #{current_user.id} scheduled a deletion of group ID #{group.id} with job ID #{job_id}")
end end
# rubocop: disable CodeReuse/ActiveRecord # rubocop: disable CodeReuse/ActiveRecord
......
...@@ -20,7 +20,7 @@ module Labels ...@@ -20,7 +20,7 @@ module Labels
label.save label.save
label label
else else
Rails.logger.warn("target_params should contain :project or :group or :template, actual value: #{target_params}") # rubocop:disable Gitlab/RailsLogger Gitlab::AppLogger.warn("target_params should contain :project or :group or :template, actual value: #{target_params}")
end end
end end
end end
......
---
title: Use applogger in groups destroy service and label create service
merge_request: 32195
author: Rajendra Kadam
type: fixed
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