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

ref(proguard): Use existing chunked upload logic #2318

Merged
merged 2 commits into from
Dec 18, 2024

Conversation

szokeasaurusrex
Copy link
Member

Use the existing chunked upload logic (already used for DIFs) when chunked-uploading Proguard mappings.

Closes #2195, #2196

@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-13 branch from ceef75b to 369a57e Compare December 16, 2024 16:51
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-12 branch from 3c8f73c to 0c48e66 Compare December 17, 2024 20:48
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-13 branch from 369a57e to 4a6ea5d Compare December 18, 2024 13:18
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-12 branch from e6f01fa to 27c7296 Compare December 18, 2024 13:25
We should not mention `upload-dif` in this error message because (1) `upload-dif` is a soft-deprecated hidden alias to `debug-files upload` and (2) `poll_assemble` has now been generalized and could be used with other commands in the future.

For reason (2), this change makes the error message general rather than referring to the `debug-files upload` command.
Use the existing chunked upload logic (already used for DIFs) when chunked-uploading Proguard mappings.

Closes #2195, #2196
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-13 branch from 4a6ea5d to 086d139 Compare December 18, 2024 13:26
Base automatically changed from szokeasaurusrex/proguard-upload-12 to master December 18, 2024 13:38
@szokeasaurusrex szokeasaurusrex merged commit 086d139 into master Dec 18, 2024
13 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/proguard-upload-13 branch December 18, 2024 13:41
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.

Generalize chunk uploading logic used to upload debug files
2 participants