Clean up non TODO rubocop errors

2 files changed, 5 insertions, 5 deletions

diff --git a/lib/gitlab/checks/change_access.rb b/lib/gitlab/checks/change_access.rb
index 6f574a41727..07fd4024346 100644
--- a/lib/gitlab/checks/change_access.rb
+++ b/lib/gitlab/checks/change_access.rb
@@ -71,15 +71,15 @@ module Gitlab
       def protected_tag_checks
         return unless tag_protected?
 
-        if forced_push?
-          return "You are not allowed to force push protected tags." #TODO: Wording, 'not allowed to update proteted tags'?
+        if forced_push? #TODO: Verify if this should prevent all updates, and mention in UI and documentation
+          return "Protected tags cannot be updated."
         end
 
         if Gitlab::Git.blank_ref?(@newrev)
-          return "You are not allowed to delete protected tags." #TODO: Wording, do these need to mention 'you' if the rule applies to everyone
+          return "Protected tags cannot be deleted."
         end
 
-        if !user_access.can_push_tag?(@tag_name)
+        unless user_access.can_push_tag?(@tag_name)
           return "You are not allowed to create this tag as it is protected."
         end
       end
diff --git a/lib/gitlab/import_export/relation_factory.rb b/lib/gitlab/import_export/relation_factory.rb
index d44563333a5..9d269c5d384 100644
--- a/lib/gitlab/import_export/relation_factory.rb
+++ b/lib/gitlab/import_export/relation_factory.rb
@@ -7,7 +7,7 @@ module Gitlab
                     triggers: 'Ci::Trigger',
                     builds: 'Ci::Build',
                     hooks: 'ProjectHook',
-                    merge_access_levels: 'ProtectedBranch::MergeAccessLevel',
+                    merge_access_levels: 'ProtectedBranch::MergeAccessLevel', #TODO: Tags
                     push_access_levels: 'ProtectedBranch::PushAccessLevel',
                     labels: :project_labels,
                     priorities: :label_priorities,