Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

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

Already on GitHub? # to your account

Error out on unexpected command line argument #207

Merged
merged 1 commit into from
Jan 12, 2023

Conversation

MichaelEischer
Copy link
Member

What is the purpose of this change? What does it change?

rest-server doesn't accept arguments. Thus, error out to prevent wrong usage.

Was the change discussed in an issue or in the forum before?

Fixes #206

Checklist

  • I have enabled maintainer edits for this PR
  • I have added tests for all changes in this PR
  • I have added documentation for the changes (in the manual)
  • There's a new file in changelog/unreleased/ that describes the changes for our users (template here)
  • I have run gofmt on the code in all commits
  • All commit messages are formatted in the same style as the other commits in the repo
  • I'm done, this Pull Request is ready for review

Copy link
Contributor

@wojas wojas left a comment

Choose a reason for hiding this comment

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

LGTM

rest-server doesn't accept arguments. Thus, error out to prevent wrong
usage.
@MichaelEischer
Copy link
Member Author

I've changed the changelog entry type to "Change" as it is more of a breaking change than a bugfix.

@MichaelEischer MichaelEischer merged commit 7f29dcb into restic:master Jan 12, 2023
@MichaelEischer MichaelEischer deleted the err-on-args branch January 12, 2023 21:00
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

--private-repos flag is being ignored in docker image
2 participants