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.

Sign up for GitHub

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

Jump to bottom

Fix ARGS parsing #67

Merged

dmolina merged 4 commits into dmolina:master from BKaperick:fix_argparsing

Dec 17, 2023

Contributor

BKaperick commented Dec 17, 2023

Proposal to fix Issue #66, I have added some basic tests as well.

BKaperick added 4 commits

December 17, 2023 11:26


          Add parse_arguments function and replace the naive split on space

a58e161


          We also need to wrap each arg in quotes when sent to socket

6eaa259


          Added test

ae21596


          Clean

1e41546

dmolina merged commit a1942e2 into dmolina:master

Owner

dmolina commented Dec 17, 2023

Thank you a lot for your proposal, merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet