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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass showProgress to fetchOptions #1454

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

riseshia
Copy link

@riseshia riseshia commented Sep 4, 2023

Fix: #1453

#1067 seems not to pass showProgress to fetchOptions, which makes any fetch call handled without --progress
Confirmed works correctly with default / explicitly given show-progress option, here is test result
https://github.com/riseshia/actions-playground/actions/runs/6078541181

I don't add test for this, because I couldn't find proper place for this case(most test are unit testing, but this problem is from "passing" arg) 馃
any suggestion for adding test is welcome. ;)

@riseshia
Copy link
Author

riseshia commented Jan 8, 2024

Conflict resolved. Could anybody check this?
馃摑 diff for #1455 is not included, we might handle this seperately.

@riseshia riseshia requested a review from takost January 8, 2024 06:01
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.

v4 with default config (no with) does not include --progress flag
2 participants