Yes giving connection strings as URL is standard in Influxdb. But it would diverge from conventions used for other database hooks and can cause confusion for users familiar with other hooks.
If…
Thanks for your feedback @witten. I will make the suggested changes. I am sorry that my response is a bit late but I had a very hectic week at university, so couldnt start before.
929d343214
Add CLI flags for every config option and add config options for many action flags (#303).
9ea55d9aa3
Add a documentation note about a limitation: You can't pass flags as values to flags (#303).
3eabda45f2
If a boolean option name already starts with "no_", don't add a "--no-no-..." CLI flag (#303).
I think u are right. I have made changes accordingly in final commit.
Thank you. I have created some differentiation from earlier tests for the last test by testing only for slot in yubikey string value so that you might be inclined to accept it 😄
The latest commit generates command as per documentation.All tests were successful.Please let me know if any more changes are required.
Ok I will get right to it !!
@witten I checked with KiaraGrouwstra, and she has given me permission to take over this issue/PR. Their last update mentioned that the next step was running the tests, so I'll start from there…
Added all suggested changes.Please review.
I would like to work on this issue. What remains to be implemented and what should I do as I can see a PR already for this issue??
I am not sure if yubikey option should be implemented as boolean flag or not. But on googling,I found that when modeling YubiKey usage in a configuration file or schema, it might be represented as…