-
Notifications
You must be signed in to change notification settings - Fork 12
WIP: Lime-App integration #60
base: master
Are you sure you want to change the base?
Conversation
…ncing, and changed name to note
end | ||
|
||
local function add_member_voucher(msg) | ||
local hostnameOutput = shell("cat /proc/sys/kernel/hostname") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks unnecesary long... could be splitted in smaller functions that contain part of the functionalty? looks it has many responsabilities
Please merge this, for all that is most sacred pirania is very important for community networks to maintain and have a minimum of security in their virtual environments. |
Everyone wants this to be merged, but it still needs some work to get healthy enough to be part of the trunk. |
🚧 🚧 🚧