Skip to content
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

Add new option to print out data saving information #96

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

HappyTobi
Copy link

Hi,

i added a new feature to simple print out the saved bytes.
You can see an example at the README.md i added an example log and the complete description to that.

I also bumped the version for you to version 2.4.3

Hope you like it.

Kind regards.

@HappyTobi
Copy link
Author

@Klathmon what do you think?

Klathmon added a commit that referenced this pull request Oct 17, 2019
@Klathmon
Copy link
Owner

@HappyTobi I like the idea a lot! I made a few changes here to some things and merged it with the changes that came from another PR (an option to enable or disable the "only use if image gets smaller" behaviour).

If you can give me permissions to commit to your branch of this repo, I can push my changes up to yours and merge this PR so you get credit for #hacktoberfest!

But until then, the changes I made are living at the sizeInfo branch here.

@HappyTobi
Copy link
Author

HappyTobi commented Oct 18, 2019

Thx for the response!
I will resolve the conflicts.

Kind regards

@HappyTobi
Copy link
Author

I merged and fix the issues.

@HappyTobi
Copy link
Author

Will the pr be merged?

@RhysTowey
Copy link

RhysTowey commented Nov 13, 2020

@Klathmon Any update on this?

@DaveSeidman
Copy link

I'd love to see this feature added as well.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants