tidy: check fluent files for style
This commit is contained in:
parent
f241c0c43d
commit
00fec76ab5
3 changed files with 11 additions and 14 deletions
|
@ -125,16 +125,13 @@ fn should_ignore(line: &str) -> bool {
|
||||||
|
|
||||||
/// Returns `true` if `line` is allowed to be longer than the normal limit.
|
/// Returns `true` if `line` is allowed to be longer than the normal limit.
|
||||||
fn long_line_is_ok(extension: &str, is_error_code: bool, max_columns: usize, line: &str) -> bool {
|
fn long_line_is_ok(extension: &str, is_error_code: bool, max_columns: usize, line: &str) -> bool {
|
||||||
if extension != "md" || is_error_code {
|
match extension {
|
||||||
if line_is_url(is_error_code, max_columns, line) || should_ignore(line) {
|
// fluent files are allowed to be any length
|
||||||
return true;
|
"ftl" => true,
|
||||||
}
|
|
||||||
} else if extension == "md" {
|
|
||||||
// non-error code markdown is allowed to be any length
|
// non-error code markdown is allowed to be any length
|
||||||
return true;
|
"md" if !is_error_code => true,
|
||||||
|
_ => line_is_url(is_error_code, max_columns, line) || should_ignore(line),
|
||||||
}
|
}
|
||||||
|
|
||||||
false
|
|
||||||
}
|
}
|
||||||
|
|
||||||
enum Directive {
|
enum Directive {
|
||||||
|
@ -230,7 +227,7 @@ pub fn check(path: &Path, bad: &mut bool) {
|
||||||
super::walk(path, &mut skip, &mut |entry, contents| {
|
super::walk(path, &mut skip, &mut |entry, contents| {
|
||||||
let file = entry.path();
|
let file = entry.path();
|
||||||
let filename = file.file_name().unwrap().to_string_lossy();
|
let filename = file.file_name().unwrap().to_string_lossy();
|
||||||
let extensions = [".rs", ".py", ".js", ".sh", ".c", ".cpp", ".h", ".md", ".css"];
|
let extensions = [".rs", ".py", ".js", ".sh", ".c", ".cpp", ".h", ".md", ".css", ".ftl"];
|
||||||
if extensions.iter().all(|e| !filename.ends_with(e)) || filename.starts_with(".#") {
|
if extensions.iter().all(|e| !filename.ends_with(e)) || filename.starts_with(".#") {
|
||||||
return;
|
return;
|
||||||
}
|
}
|
||||||
|
|
Loading…
Add table
Add a link
Reference in a new issue