🐛 fix(cli): correct error mapping in add_cli
- fix error mapping for message age creation - use direct error propagation instead of config error mapping
This commit is contained in:
committed by
Jeremiah Russell
parent
1448c791d9
commit
5c2124ead4
@@ -41,8 +41,7 @@ pub struct AddCli {
|
|||||||
impl AddCli {
|
impl AddCli {
|
||||||
pub fn run(&self, mut config: Rules) -> Result<(), Error> {
|
pub fn run(&self, mut config: Rules) -> Result<(), Error> {
|
||||||
let generate = self.label.is_none();
|
let generate = self.label.is_none();
|
||||||
let message_age = MessageAge::new(self.period.to_string().as_str(), self.count)
|
let message_age = MessageAge::new(self.period.to_string().as_str(), self.count)?;
|
||||||
.map_err(|e| Error::Config(config::ConfigError::Message(e)))?;
|
|
||||||
let retention = Retention::new(message_age, generate);
|
let retention = Retention::new(message_age, generate);
|
||||||
|
|
||||||
config.add_rule(retention, self.label.as_ref(), self.delete);
|
config.add_rule(retention, self.label.as_ref(), self.delete);
|
||||||
|
|||||||
Reference in New Issue
Block a user