From 18a057af9fdc8a24255218718846c18268b7a2d6 Mon Sep 17 00:00:00 2001 From: rszyma Date: Wed, 2 Aug 2023 11:56:23 +0200 Subject: [PATCH] Apply suggestions from code review (1/2) Co-authored-by: jtroo --- parser/src/cfg/mod.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/parser/src/cfg/mod.rs b/parser/src/cfg/mod.rs index 967a784a0..eb3efca99 100644 --- a/parser/src/cfg/mod.rs +++ b/parser/src/cfg/mod.rs @@ -124,7 +124,7 @@ macro_rules! anyhow_span { } pub struct FileContentProvider<'a> { - /// An function to load content of a file from a filepath. + /// A function to load content of a file from a filepath. /// Optionally, it could implement caching and a mechanism preventing "file" and "./file" from loading twice. get_file_content_fn: &'a mut dyn FnMut(&Path) -> std::result::Result, } @@ -271,7 +271,7 @@ fn parse_cfg_raw( KeySeqsToFKeys, Overrides, )> { - const INVALID_PATH_ERROR: &str = "The provided config file path is not a valid"; + const INVALID_PATH_ERROR: &str = "The provided config file path is not valid"; // IDEA: Maybe store the actual content of the files here // and get rid of `file_content` in [`sexpr::Span`]?