Skip to content

Remove full stop from log line to be consistent with all other log lines#364

Merged
rawtaz merged 1 commit intorestic:masterfrom
mattxtaz:master
Dec 7, 2025
Merged

Remove full stop from log line to be consistent with all other log lines#364
rawtaz merged 1 commit intorestic:masterfrom
mattxtaz:master

Conversation

@mattxtaz
Copy link
Contributor

@mattxtaz mattxtaz commented Oct 5, 2025

What does this PR change? What problem does it solve?

Just a simple style change to be consistent.

Was the change previously discussed in an issue or on the forum?

Mentioned on IRC.

Checklist

  • I have added tests for all code changes.
  • I have added documentation for relevant changes (in the manual).
  • There's a new file in changelog/unreleased/ that describes the changes for our users (see template).
  • I'm done! This pull request is ready for review.

Copy link
Contributor

@rawtaz rawtaz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't argue with this 😅

@rawtaz rawtaz enabled auto-merge December 3, 2025 17:56
@rawtaz rawtaz disabled auto-merge December 3, 2025 18:36
@rawtaz rawtaz enabled auto-merge December 3, 2025 18:36
@MichaelEischer MichaelEischer marked this pull request as draft December 6, 2025 19:06
auto-merge was automatically disabled December 6, 2025 19:06

Pull request was converted to draft

@MichaelEischer MichaelEischer marked this pull request as ready for review December 6, 2025 19:06
@MichaelEischer
Copy link
Member

@mattxtaz Apparently we were to slow to trigger the CI for the PR and now the button to allow the CI run has disappeared. Please modify the commit (easiest way is to use git commit --amend which will just change the commit date) and the force push to this branch.

@rawtaz rawtaz enabled auto-merge December 7, 2025 18:53
@rawtaz rawtaz merged commit 4e4e8c0 into restic:master Dec 7, 2025
5 checks passed
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.

3 participants