Update CheckForWindowsLineEndings to check JS/TS/CSS/HTML/SVG files.
Bug: None Change-Id: Ic74cbf5585bf17b7f6b565878c28c81b4b3ca22d Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/6256276 Commit-Queue: Demetrios Papadopoulos <dpapad@chromium.org> Auto-Submit: Demetrios Papadopoulos <dpapad@chromium.org> Reviewed-by: Dirk Pranke <dpranke@google.com> Cr-Commit-Position: refs/heads/main@{#1419624}
This commit is contained in:
12
PRESUBMIT.py
12
PRESUBMIT.py
@@ -6461,13 +6461,21 @@ def CheckForWindowsLineEndings(input_api, output_api):
|
|||||||
"""
|
"""
|
||||||
known_text_files = r'.*\.(txt|html|htm|py|gyp|gypi|gn|isolate|icon)$'
|
known_text_files = r'.*\.(txt|html|htm|py|gyp|gypi|gn|isolate|icon)$'
|
||||||
|
|
||||||
|
_WEBUI_FILES_EXTENSIONS = r'\.(css|html|js|ts|svg)$'
|
||||||
|
|
||||||
file_inclusion_pattern = (known_text_files,
|
file_inclusion_pattern = (known_text_files,
|
||||||
r'.+%s' % _IMPLEMENTATION_EXTENSIONS,
|
r'.+%s' % _IMPLEMENTATION_EXTENSIONS,
|
||||||
r'.+%s' % _HEADER_EXTENSIONS)
|
r'.+%s' % _HEADER_EXTENSIONS,
|
||||||
|
r'.+%s' % _WEBUI_FILES_EXTENSIONS)
|
||||||
|
|
||||||
|
# Exclude folder that contains .ts files that are actually binary video
|
||||||
|
# format and not TypeScript.
|
||||||
|
file_exclusion_pattern = (r'media/test/data/')
|
||||||
|
|
||||||
problems = []
|
problems = []
|
||||||
source_file_filter = lambda f: input_api.FilterSourceFile(
|
source_file_filter = lambda f: input_api.FilterSourceFile(
|
||||||
f, files_to_check=file_inclusion_pattern, files_to_skip=None)
|
f, files_to_check=file_inclusion_pattern,
|
||||||
|
files_to_skip=file_exclusion_pattern)
|
||||||
for f in input_api.AffectedSourceFiles(source_file_filter):
|
for f in input_api.AffectedSourceFiles(source_file_filter):
|
||||||
# Ignore test files that contain crlf intentionally.
|
# Ignore test files that contain crlf intentionally.
|
||||||
if f.LocalPath().endswith('crlf.txt'):
|
if f.LocalPath().endswith('crlf.txt'):
|
||||||
|
Reference in New Issue
Block a user