Revert "audit: add rules for non-ascii character and encoding comment"
This reverts commit ab8fe812377bc60cc18fe8d7c1ea8ce2f0269294.
This commit is contained in:
parent
7d8bec3f20
commit
d3ab5e6034
@ -74,14 +74,6 @@ class FormulaText
|
|||||||
/\Z\n/ =~ @text
|
/\Z\n/ =~ @text
|
||||||
end
|
end
|
||||||
|
|
||||||
def has_non_ascii_character?
|
|
||||||
/[^\x00-\x7F]/ =~ @text
|
|
||||||
end
|
|
||||||
|
|
||||||
def has_encoding_comment?
|
|
||||||
/^# (en)?coding: utf-8$/i =~ @text
|
|
||||||
end
|
|
||||||
|
|
||||||
def =~ regex
|
def =~ regex
|
||||||
regex =~ @text
|
regex =~ @text
|
||||||
end
|
end
|
||||||
@ -130,14 +122,6 @@ class FormulaAuditor
|
|||||||
problem "'__END__' was found, but 'DATA' is not used"
|
problem "'__END__' was found, but 'DATA' is not used"
|
||||||
end
|
end
|
||||||
|
|
||||||
if text.has_non_ascii_character? and not text.has_encoding_comment?
|
|
||||||
problem "Found non-ASCII character: add `# encoding: UTF-8` in the first line"
|
|
||||||
end
|
|
||||||
|
|
||||||
if text.has_encoding_comment? and not text.has_non_ascii_character?
|
|
||||||
problem "Remove the redundant `# encoding: UTF-8`"
|
|
||||||
end
|
|
||||||
|
|
||||||
unless text.has_trailing_newline?
|
unless text.has_trailing_newline?
|
||||||
problem "File should end with a newline"
|
problem "File should end with a newline"
|
||||||
end
|
end
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user