-
Notifications
You must be signed in to change notification settings - Fork 34
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
Cleanup #636
Cleanup #636
Conversation
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.
As this diverges from the SLE15 branch the version should be updated to 4.1.0
.
Apart of that, the methods that are left are only used by the inst_do_net_test.rb client, that is only used for examples https://github.com/search?p=1&q=org%3Ayast+do_net_test&type=Code What about droping it? |
dropping inst_do_net_test is the goal in this case, I'm still searching (and cherry-picking from my "refactoring" branch) for more code to remove -> less code to refactor ;-)) |
package/yast2-network.changes
Outdated
------------------------------------------------------------------- | ||
Wed Jun 20 07:24:24 UTC 2018 - mfilka@suse.com | ||
|
||
- dropped useless and / or dead code |
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.
NP: dead code is usually useless, dropped dead code
could be enough IMHO...
Dropped some unreferenced methods, routing dialog definition into smaller parts