search

Vladimir-csp / uwsm

Universal Wayland Session Manager
MIT License
149 stars 6 forks source link

want to add some doctest? #6

Open skewballfox opened 1 year ago

skewballfox commented 1 year ago

I was thinking about adding doctest for a few of these functions. it's part of the standard library so won't incur an extra dependency.

Some of these functions would be hard to make sense of without some examples of input, and some testing is probably necessary as right now the only way to confirm it works is to well run it. given that some of the capabilities likely require sudo that is probably an undesirable way to discover bugs

Vladimir-csp commented 1 year ago

Go ahead. FYI, I'm nuking wrap_process soon. None of the capabilities require root privileges.