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
Documentation: update the docs strings #126
Conversation
69f1589
to
b3eb48f
Compare
b3eb48f
to
7c2cea2
Compare
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.
Repeating answer, so only commented them once :)
7c2cea2
to
e651b37
Compare
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.
You can even consider stricter types, like Stdlib::IP::Address::Nosubnet
and Stdlib::Fqdn
for return types, but this is already an improvement.
It does look like the tests uncovered some issues though, so it looks like return_type
does validate the type as well. Which I'd consider a nice bonus :)
e651b37
to
6883561
Compare
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.
Other the failing tests, 👍
6d5058a
to
3ca9600
Compare
Also change the signature of the functions to make use of optional_block_param We also improve the type validation which requires adding a dependency on stdlib
3ca9600
to
463dbf1
Compare
Also change the signature of the functions to make use of optional_block_param