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): Remove upload-dif reference from poll_assemble #2321

Merged
merged 1 commit into from
Dec 18, 2024

Conversation

szokeasaurusrex
Copy link
Member

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.

@loewenheim
Copy link
Contributor

This PR has a huge diff, is that intentional?

@szokeasaurusrex
Copy link
Member Author

@loewenheim No, looks like there are some extra commits in here (the extra commits should be in the PRs that this change depends on). Might need to rebase and update some things. I'll ping you once I've fixed it

@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-12a branch from 590f24d to f6d2555 Compare December 18, 2024 13:17
@szokeasaurusrex
Copy link
Member Author

@loewenheim fixed it, thanks for pointing out the big diff!

@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.
@szokeasaurusrex szokeasaurusrex force-pushed the szokeasaurusrex/proguard-upload-12a branch from f6d2555 to 3fd0566 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 3fd0566 into master Dec 18, 2024
13 checks passed
@szokeasaurusrex szokeasaurusrex deleted the szokeasaurusrex/proguard-upload-12a branch December 18, 2024 13:38
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