Skip to content

CompressArchiveCommand: fix issues with progress formatting#143

Open
ForNeVeR wants to merge 1 commit intoPowerShell:masterfrom
ForNeVeR:improvement
Open

CompressArchiveCommand: fix issues with progress formatting#143
ForNeVeR wants to merge 1 commit intoPowerShell:masterfrom
ForNeVeR:improvement

Conversation

@ForNeVeR
Copy link

@ForNeVeR ForNeVeR commented Dec 7, 2022

PR Summary + PR Context

  • The progress now is actually reported (I guess missing $ was a typo).
  • The progress now uses current culture (that's why I've replaced hard-coded 100.0 with $"{100.0:0.0}": I assume the idea was to use local culture to format the number; certain cultures may use different number separator and formatting).

@ForNeVeR
Copy link
Author

ForNeVeR commented Dec 7, 2022

I can see that this change is partially included in #140, and yet 100.0 is always formatted in a culture-invariant way there.

- The progress now is actually reported.
- The progress now uses current culture.
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.

1 participant