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

Fix support for -dash options #32

Merged
merged 1 commit into from May 18, 2017

Conversation

Projects
None yet
2 participants
@choroba
Contributor

choroba commented May 18, 2017

Previously, the code exited before the dash was removed from options.
The output pdf table was fixed, too, now it's the same as generated
from the code without the dashes.

Fix support for -dash options
Previously, the code exited before the dash was removed from options.
The output pdf table was fixed, too, now it's the same as generated
from the code without the dashes.
@choroba

This comment has been minimized.

Show comment
Hide comment
@choroba

choroba May 18, 2017

Contributor

Also, what's the point of keeping the generated pdf file in the repository?

Contributor

choroba commented May 18, 2017

Also, what's the point of keeping the generated pdf file in the repository?

@kamenov

This comment has been minimized.

Show comment
Hide comment
@kamenov

kamenov May 18, 2017

Owner

Thanks for contributing. Will release your changes to CPAN by the end of this week.

The point is that you have the results before you setup and try out the project so that you can see if the result will be what you are looking for.
Generally it's questionable how useful is that but they are small enough to stay for now.

Owner

kamenov commented May 18, 2017

Thanks for contributing. Will release your changes to CPAN by the end of this week.

The point is that you have the results before you setup and try out the project so that you can see if the result will be what you are looking for.
Generally it's questionable how useful is that but they are small enough to stay for now.

@kamenov kamenov merged commit 44fca9c into kamenov:master May 18, 2017

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