-
Notifications
You must be signed in to change notification settings - Fork 18
Open
Description
Hello, thank you for this helper classes. Made my task much easier.
Howevery, I have an improvement suggestion: registerService should have an overloading with port parameter specified by the caller.
Another suggestion: Don't use package-local as default for such libraries, but protected. This let us modify the behaviour more easy in the projects. In this case, if findAvaiablePort() was declared protected, I could have overriden it myself in a subclass.
Thanks. :-)
EDIT: I've just seen it was added in November, however, the package 0.1.4 in Jitpack seems to be older.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels