Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

net: provide list of available nets #61

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

blueyed
Copy link
Contributor

@blueyed blueyed commented Oct 3, 2018

This makes it easier to loop over them and/or handle them separately.

This makes it easier to loop over them and/or handle them separately.
@blueyed
Copy link
Contributor Author

blueyed commented Oct 4, 2018

Should also be done for ./widgets/net_freebsd.lua then likely.

@McSinyx
Copy link
Member

McSinyx commented Oct 22, 2018

Documentation needs update. Although as a Python guy I think explicit is better than implicit, so maybe we should take the approach currently used in net_freebsd widget, i.e. match for '^' .. name .. ':' instead.

Edit: The explicit approach breaks backward compatibility however.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants