Closed newtoallofthis123 closed 3 months ago
looks good. lgtm
Thanks!
Hi, I didn't get a chance to test this pr @zubairmh Did you do so?
Added some stuff that would enable the environ.rs
to check if the path provided is valid or not
This looks good to me. I haven't gotten around to testing it yet. @newtoallofthis123 Have you tested the PR?
I did test it a while ago and everything seemed to work fine 😄
However, I'll test it again tomorrow and let you know just to be safe :)
@newtoallofthis123 looks good to go, have you tested this?
Yep! tested it and it works :)
This change introduces similar changes as #243 for
swhkd
.The following were the changes made:
environ.rs
which is central to the environment management systemenv::var
uses across thedaemon.rs