bjoernager/rust
bjoernager/rust
1
Fork 0
Code Activity

add FIXME note in TomlConfig::merge

Browse source 
Signed-off-by: onur-ozkan <work@onurozkan.dev>
This commit is contained in:
onur-ozkan 2025-04-15 11:53:21 +03:00
parent 8e6f50bb4d
commit 7dfb457745
1 changed files with 2 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

2
src/bootstrap/src/core/config/config.rs
View file

@ -812,6 +812,8 @@ impl Merge for TomlConfig {
exit!(2); exit!(2);
}); });
// FIXME: Similar to `Config::parse_inner`, allow passing a custom `get_toml` from the caller to
// improve testability since `Config::get_toml` does nothing when `cfg(test)` is enabled.
let included_toml = Config::get_toml(&include_path).unwrap_or_else(|e| { let included_toml = Config::get_toml(&include_path).unwrap_or_else(|e| {
eprintln!("ERROR: Failed to parse '{}': {e}", include_path.display()); eprintln!("ERROR: Failed to parse '{}': {e}", include_path.display());
exit!(2); exit!(2);