Apply suggestions from code review

Co-authored-by: rawtaz <rawtaz@users.noreply.github.com>
This commit is contained in:
Michael Eischer
2026-06-24 19:26:51 +02:00
committed by Michael Eischer
parent f058670219
commit fc422d4145
5 changed files with 17 additions and 15 deletions
+4 -4
View File
@@ -1,8 +1,8 @@
Bugfix: Hide progress bar for `stats` in JSON mode
Bugfix: Hide `stats` progress bar in JSON mode
Since restic 0.19.0, the `stats` command shows a progress bar. However,
it was also active with the `--json` option, which mixed text output with
JSON. This has been fixed.
Since restic 0.19.0, the `stats` command shows a progress bar. This
progress bar was unintentionally displayed also when using the `--json`
option, mixing regular text output with JSON. This is now fixed.
https://github.com/restic/restic/issues/21866
https://github.com/restic/restic/pull/21871
+6 -5
View File
@@ -1,11 +1,12 @@
Bugfix: Restore old behavior of `snapshots --latest <n>` without `--group-by`
Restic 0.19.0 changed the behavior of `snapshots --latest <n>` to no longer
group snapshots by default.
Restic 0.19.0 accidentally changed the behavior of `snapshots --latest <n>`
to no longer group snapshots by host and paths by default.
`snapshots --latest <n>` again uses the old behavior when `--group-by` is not
specified. With `--group-by`, the output is still grouped as requested, as in
restic 0.19.0.
The `snapshots --latest <n>` command now again uses the old behavior of
grouping by host and paths when `--group-by` is not specified. However, when
specifying `--group-by` the output is still grouped as requested, as in restic
0.19.0.
https://github.com/restic/restic/issues/21869
https://github.com/restic/restic/pull/21875
+2 -2
View File
@@ -1,8 +1,8 @@
Bugfix: Respect excludes for duplicate directory entries during `backup`
Bugfix: Make `backup` respect excludes for duplicate directory entries
Since restic 0.19.0, backing up a directory with duplicate directory
entries always produced "Warning: at least one source file could not be
read" even when those files were excluded. This has been fixed.
read", even when those files were excluded. This has now been fixed.
https://github.com/restic/restic/issues/21899
https://github.com/restic/restic/pull/21900
+4 -3
View File
@@ -1,8 +1,9 @@
Bugfix: Show timezone location in `snapshots` output
Since restic 0.19.0, the `snapshots` command printed the current timezone when
listing snapshots. That label can change during the year, for example with
daylight saving time. Restic now prints the location name instead.
With restic 0.19.0, the `snapshots` command printed the current timezone when
listing snapshots. However, that timezone label might change during the year,
for example with daylight saving time. Restic now prints a more consistent and
shorter version of the text.
https://github.com/restic/restic/pull/21876
https://forum.restic.net/t/possible-bug-in-timezone-naming/10867
+1 -1
View File
@@ -2,6 +2,6 @@ Bugfix: Prevent crash in mountpoint validation if mountpoint is inaccessible
Since restic 0.19.0, the `mount` command validates a mountpoint before loading
the repository. If restic was unable to stat the mountpoint, this would result
in a crash. This has been fixed to correctly return an error instead.
in a crash. This has now been fixed to correctly return an error instead.
https://github.com/restic/restic/pull/21879