-
Notifications
You must be signed in to change notification settings - Fork 3
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
os:cmd instead of Erlang ports #23
Conversation
Change is to be discussed. There must be a rebar3 facility one can use to output e.g. with colors
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.
Looks great. Thank you for all the cleanup you did. This was the first ever thing I made in Erlang and it was due for some love.
I pushed a fix for the tests. I also ended up having |
Hurrah, tests passed. Edit: you can see how it looks here. |
nicely done. I'll merge this now and anything that you like to improve on can be separate PRs. |
If you feel it's OK, could you release it and possibly publish it to Hex? Thanks. |
Attempting to [Fix #16] [Fix #21] [Fix #22], by replacing Erlang ports with
os:cmd
as discussed in two of the linked Issues.