foster/denoland-deno
1
0
Fork 0
mirror of https://github.com/denoland/deno.git synced 2024-11-25 15:29:32 -05:00
Code Issues Packages 1 Wiki Activity

refactor(cli): simplify lint/format resolver logic (#12898)

Browse source
This commit is contained in:
Zheyu Zhang 2021-11-29 22:17:57 +08:00 committed by Luca Casonato
parent 4acb46e3d9
commit 1333334496
2 changed files with 39 additions and 42 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

49
cli/tools/fmt.rs
View file

@ -87,35 +87,34 @@ pub async fn format(
let resolver = |changed: Option<Vec<PathBuf>>| {
let files_changed = changed.is_some();
let collect_files =
collect_files(&include_files, &exclude_files, is_supported_ext_fmt);
let (result, should_refmt) = match collect_files {
Ok(value) => {
if let Some(paths) = changed {
let refmt_files = value
.clone()
.into_iter()
.filter(|path| paths.contains(path))
.collect::<Vec<_>>();
let should_refmt = !refmt_files.is_empty();
if check {
(Ok((value, fmt_options.clone())), Some(should_refmt))
let result =
collect_files(&include_files, &exclude_files, is_supported_ext_fmt).map(
|files| {
let refmt_files = if let Some(paths) = changed {
if check {
files
.iter()
.any(|path| paths.contains(path))
.then(|| files)
.unwrap_or_else(|| [].to_vec())
} else {
files
.into_iter()
.filter(|path| paths.contains(path))
.collect::<Vec<_>>()
}
} else {
(Ok((refmt_files, fmt_options.clone())), Some(should_refmt))
}
} else {
(Ok((value, fmt_options.clone())), None)
}
}
Err(e) => (Err(e), None),
};
files
};
(refmt_files, fmt_options.clone())
},
);
let paths_to_watch = include_files.clone();
async move {
if files_changed && matches!(should_refmt, Some(false)) {
if files_changed
&& matches!(result, Ok((ref files, _)) if files.is_empty())
{
ResolutionResult::Ignore
} else {
ResolutionResult::Restart {

32
cli/tools/lint.rs
View file

@ -112,27 +112,25 @@ pub async fn lint(
let resolver = |changed: Option<Vec<PathBuf>>| {
let files_changed = changed.is_some();
let collect_files =
collect_files(&include_files, &exclude_files, is_supported_ext);
let result =
collect_files(&include_files, &exclude_files, is_supported_ext).map(
|files| {
if let Some(paths) = changed {
files
.iter()
.any(|path| paths.contains(path))
.then(|| files)
.unwrap_or_else(|| [].to_vec())
} else {
files
}
},
);
let paths_to_watch = include_files.clone();
let (result, should_relint) = match collect_files {
Ok(value) => {
if let Some(paths) = changed {
(
Ok(value.clone()),
Some(value.iter().any(|path| paths.contains(path))),
)
} else {
(Ok(value), None)
}
}
Err(e) => (Err(e), None),
};
async move {
if files_changed && matches!(should_relint, Some(false)) {
if files_changed && matches!(result, Ok(ref files) if files.is_empty()) {
ResolutionResult::Ignore
} else {
ResolutionResult::Restart {