jpictl: fix cloud.yaml unmarshalling #32
Merged
amery
merged 2 commits from pr-amery-unmarshaltext
into main
1 year ago
Loading…
Reference in new issue
There is no content yet.
Delete Branch 'pr-amery-unmarshaltext'
Deleting a branch is permanent. It CANNOT be undone. Continue?
fix cloud.yaml unmarshalling by implementing UnmarshalText on wireguard endpoint addresses and keys
Looks a bit strange to me to have
switch
statement with only 2 cases from which one isdefault
, but otherwise LGTMI thought you preferred switch over if/else @karasz
This would have been an
for anything that has more than 2 branches I prefer
switch
as it seems to me more appropriate .@karasz amended accordingly, and rebased over the current
main
+1
5ef6d45ef7
into main 1 year agoReviewers
5ef6d45ef7
.