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

feat: add support for chunked uploading #448

Merged
merged 1 commit into from
Jul 18, 2023

Conversation

stephenc
Copy link
Contributor

tested manually with GIF images, but no automated tests in this

I hereby declare that this contribution is licensed under Apache License Version 2

@redouane59 feel free to rework however you need. Should fix #183 and #281 but is hacky and not my best code

tested manually with GIF images, but no automated tests in this

I hereby declare that this contribution is licensed under Apache License Version 2
@sonarcloud
Copy link

sonarcloud bot commented Jul 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug C 1 Bug
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell B 46 Code Smells

1.5% 1.5% Coverage
0.0% 0.0% Duplication

@redouane59
Copy link
Owner

redouane59 commented Jul 17, 2023

Hello,
Thanks ! Can you just correct the little bug that sonar shows ?

@stephenc
Copy link
Contributor Author

@redouane59 feel free to rework however you need

Just driving by my friend. But getting a release that can let me tweet gifs would be much appreciated, do whatever you want with the PR

@stephenc
Copy link
Contributor Author

To clarify, I don’t have the headspace to determine what the idiomatic way of handling the interrupted exception is in your library. Probably quicker if you just push the way you want it on top. This PR should allow edits from maintainers, and I am happy for you to align the code with your style

@redouane59 redouane59 merged commit 05f363e into redouane59:develop Jul 18, 2023
@redouane59 redouane59 mentioned this pull request Jul 18, 2023
redouane59 added a commit that referenced this pull request Jul 18, 2023
* feat: add support for chunked uploading (#448)

tested manually with GIF images, but no automated tests in this

I hereby declare that this contribution is licensed under Apache License Version 2

* upload version

---------

Co-authored-by: Stephen Connolly <[email protected]>
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.

Gif cannot be read
2 participants