[Enhancement]: Return library error on `slurm_ops` instead of propagating `CalledProcessError`

charmed-hpc / hpc-libs

A collection of charm libraries for authoring HPC charms 📖🖋️

https://charmhub.io/hpc-libs

Apache License 2.0

0 stars 3 forks source link

[Enhancement]: Return library error on `slurm_ops` instead of propagating `CalledProcessError` #11

Closed jedel1043 closed 3 months ago

jedel1043 commented 3 months ago

Should be a pretty nice enhancement, considering that returning CalledProcessError is a bit more opaque to the user in contrast to having a nifty SlurmOpsError that is visible from the module exports.

NucciTheBoss commented 3 months ago

I had the same idea yesterday! Great minds think alike :exploding_head: