7c58055631
This Cop enforces the rule that injecting EE modules (using prepend, include, or extend) is done by placing the injection on the last line of a file, instead of somewhere in the middle. By placing these lines at the very end, merge conflicts will not happen.
47 lines
1.6 KiB
Ruby
47 lines
1.6 KiB
Ruby
# frozen_string_literal: true
|
|
|
|
module RuboCop
|
|
module Cop
|
|
# Cop that blacklists the injecting of EE specific modules anywhere but on
|
|
# the last line of a file. Injecting a module in the middle of a file will
|
|
# cause merge conflicts, while placing it on the last line will not.
|
|
class InjectEnterpriseEditionModule < RuboCop::Cop::Cop
|
|
MSG = 'Injecting EE modules must be done on the last line of this file' \
|
|
', outside of any class or module definitions'
|
|
|
|
METHODS = Set.new(%i[include extend prepend]).freeze
|
|
|
|
def_node_matcher :ee_const?, <<~PATTERN
|
|
(const (const _ :EE) _)
|
|
PATTERN
|
|
|
|
def on_send(node)
|
|
return unless METHODS.include?(node.children[1])
|
|
return unless ee_const?(node.children[2])
|
|
|
|
line = node.location.line
|
|
buffer = node.location.expression.source_buffer
|
|
last_line = buffer.last_line
|
|
|
|
# Parser treats the final newline (if present) as a separate line,
|
|
# meaning that a simple `line < last_line` would yield true even though
|
|
# the expression is the last line _of code_.
|
|
last_line -= 1 if buffer.source.end_with?("\n")
|
|
|
|
add_offense(node) if line < last_line
|
|
end
|
|
|
|
# Automatically correcting these offenses is not always possible, as
|
|
# sometimes code needs to be refactored to make this work. As such, we
|
|
# only allow developers to easily blacklist existing offenses.
|
|
def autocorrect(node)
|
|
lambda do |corrector|
|
|
corrector.insert_after(
|
|
node.source_range,
|
|
" # rubocop: disable #{cop_name}"
|
|
)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
end
|