Skip to content

Conversation

cmaglie
Copy link
Member

@cmaglie cmaglie commented Nov 16, 2022

Please check if the PR fulfills these requirements

See how to contribute

  • The PR has no duplicates (please search among the Pull Requests
    before creating one)
  • The PR follows
    our contributing guidelines
  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been added / updated (for bug fixes / features)
  • UPGRADING.md has been updated with a migration guide (for breaking changes)

What kind of change does this PR introduce?

Fix the progress bar.

What is the current behavior?

In an operation with multiple downloads (for example a core install with many tools), the progress bar is displayed only for the first download, the remainder just displays tool-X downloaded.

What is the new behavior?

The progress bar is displayed for all downloads.

Does this PR introduce a breaking change, and is titled accordingly?

No

Other information

This is a regression from #1875

@cmaglie cmaglie self-assigned this Nov 16, 2022
@cmaglie cmaglie added priority: low Resolution is a low priority type: imperfection Perceived defect in any part of project topic: CLI Related to the command line interface criticality: low Of low impact labels Nov 16, 2022
@cmaglie cmaglie merged commit 1fb5d14 into arduino:master Nov 16, 2022
@cmaglie cmaglie deleted the fix_progress_bar branch November 16, 2022 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
criticality: low Of low impact priority: low Resolution is a low priority topic: CLI Related to the command line interface type: imperfection Perceived defect in any part of project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants