Single option for kafka options
Created by: rerpha
Description of work
Replacing the two options for kafka options that are integers and strings to one single option. As CLI11 is now used no string formatting is needed to convert the integers to strings then sent off to kafka to set up brokers etc.
Issue
Closes #290 (closed)
Acceptance Criteria
Tests pass I have tested this with an .ini file with a debugger and it parses correctly so I would assume it'd do the same for key value pairs in a command line call outside of an .ini file but haven't tested this yet.
Unit Tests
No tests modified.
Other
Updated readme to show example of new command line option.
Code Review (To be filled in by the reviewer only)
-
Is the code of an acceptable quality? -
Do the changes function as described and is it robust?
Nominate for Group Code Review (Anyone can nominate it)
Indicate if you think the code should be reviewed in a Thursday code review session.
-
Recommend for group code review
Also, nominate it on the code_review Slack channel (does someone want to automate this?).