[pve-devel] [PATCH proxmox v2 02/12] notify: add api for webhook targets

Lukas Wagner l.wagner at proxmox.com
Mon Jul 22 09:32:06 CEST 2024



On  2024-07-17 17:35, Max Carrara wrote:
>> +
>> +        assert_eq!(secrets[1].name, "token".to_string());
>> +        assert_eq!(secrets[1].value, Some(encode("secret")));
>> +        assert_eq!(secrets[0].name, "token2".to_string());
>> +        assert_eq!(secrets[0].value, Some(encode("newsecret")));
>> +
>> +        // Test property deletion
>> +        update_endpoint(
>> +            &mut config,
>> +            "webhook-endpoint",
>> +            Default::default(),
>> +            Some(&[DeleteableWebhookProperty::Comment, DeleteableWebhookProperty::Secret]),
> 
> You missed a `cargo fmt` here ;)
> 


Right... Thanks! Was too used to RustRover autoformatting everything on save, which
I have not set up in my current nvim setup yet.

-- 
- Lukas




More information about the pve-devel mailing list