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.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Successfully removed locks #3929

Closed
solracsf opened this issue Sep 12, 2022 · 0 comments · Fixed by #3935
Closed

Successfully removed locks #3929

solracsf opened this issue Sep 12, 2022 · 0 comments · Fixed by #3935
Labels
category: user interface help: good first issue easy issue to get started with for beginners (both Go and restic) help: minor complexity less complex issue, good for beginners that want a little challenge yet not too complicated help: wanted type: feature enhancement improving existing features

Comments

@solracsf
Copy link

restic/cmd/restic/cmd_unlock.go

Line 54 in 74c0607

Verbosef("successfully removed locks\n")

Returns every time restic unlock is run. It should return only if there were locks to be removed.
@MichaelEischer MichaelEischer added type: feature enhancement improving existing features help: wanted category: user interface help: minor complexity less complex issue, good for beginners that want a little challenge yet not too complicated help: good first issue easy issue to get started with for beginners (both Go and restic) labels Sep 13, 2022
@miles170 miles170 mentioned this issue Sep 18, 2022
Merged
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
category: user interface help: good first issue easy issue to get started with for beginners (both Go and restic) help: minor complexity less complex issue, good for beginners that want a little challenge yet not too complicated help: wanted type: feature enhancement improving existing features
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Only display the message if there were locks to be removed miles170/restic
2 participants
@MichaelEischer @solracsf