Throw error when task config write is unsuccessfully (#3620)

This commit is contained in:
Gagan Nagaraj 2024-09-11 07:20:22 -07:00 committed by GitHub
parent 6a24510473
commit c00c0e941b
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -95,7 +95,9 @@ bool CmdConfig::setConfigVariable(const std::string& name, const std::string& va
change = true;
}
if (change) File::write(Context::getContext().config.file(), contents);
if (change)
if (!File::write(Context::getContext().config.file(), contents))
throw format("Could not write to '{1}'.", Context::getContext().config.file());
return change;
}
@ -133,7 +135,9 @@ int CmdConfig::unsetConfigVariable(const std::string& name, bool confirmation /*
if (!lineDeleted) line++;
}
if (change) File::write(Context::getContext().config.file(), contents);
if (change)
if (!File::write(Context::getContext().config.file(), contents))
throw format("Could not write to '{1}'.", Context::getContext().config.file());
if (change && found)
return 0;