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(chunks): Extract upload logic from upload_difs_chunked #2316

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

szokeasaurusrex
Copy link
Member

The upload_difs_chunked function contains logic specific to uploading DIFs, in addition to logic general to chunked uploads.

This change extracts the general chunked upload logic, so it can be reused for other types of chunked uploads.

Successor of #2303


/// Renders the given detail string to the command line. If the `detail` is
/// either missing or empty, the optional fallback will be used.
fn render_detail(detail: &Option<String>, fallback: Option<&str>) {
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This &Option<String> was copied from existing code

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will be fixed in #2317

@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-11 branch from 3c8f73c to fafb281 Compare December 16, 2024 16:27
@szokeasaurusrex
Copy link
Member Author

Note to self (reason for marking as draft): Before merging, rename variables and modify comments as needed to make any references to DIFs generic to any uploaded object

@szokeasaurusrex szokeasaurusrex marked this pull request as draft December 16, 2024 16:52
The `upload_difs_chunked` function contains logic specific to uploading DIFs, in addition to logic general to chunked uploads.

This change extracts the general chunked upload logic, so it can be reused for other types of chunked uploads.
Base automatically changed from szokeasaurusrex/proguard-upload-10 to master December 17, 2024 20:47
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-11 branch from fafb281 to 3a8b380 Compare December 17, 2024 20:47
@szokeasaurusrex szokeasaurusrex marked this pull request as ready for review December 17, 2024 20:47
@szokeasaurusrex szokeasaurusrex merged commit 3a8b380 into master Dec 18, 2024
14 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/proguard-upload-11 branch December 18, 2024 13:22
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.

2 participants