refactor: Use ? operator instead of unwrapping inside of setup()

This commit is contained in:
Kiril Kovachev 2025-05-08 23:42:08 +01:00
parent dec35b4c8f
commit 784c28af77

View File

@ -51,7 +51,7 @@ fn setup(args: SetupArgs) -> Result<(), std::io::Error> {
let config_template = std::fs::read_to_string(CONFIG_TEMPLATE_PATH).or(Err(std::io::Error::new(std::io::ErrorKind::NotFound, format!("Unable to find {}; did you execute the script from the same directory?", CONFIG_TEMPLATE_PATH))))?;
let filled_in_config = formatx!(config_template, args.api_url, args.rest_url, args.username, args.botpassword, args.oauth2_token).unwrap();
let path = get_bot_config_path();
std::fs::write(&path, filled_in_config).unwrap();
std::fs::write(&path, filled_in_config)?;
// Fix permissions on UNIX-like systems, since mwbot-rs doesn't like to read configs with loose permissions.
{
use std::os::unix::fs::PermissionsExt;