Mention placeholders for repositories in documentation; fix #369 #371

Merged
witten merged 1 commits from fpoulain/borgmatic:master into master 8 months ago
There is no content yet.
Owner

This is great. Thank you for taking the time to make a PR.. And a test! My only request is that in the example in the repositories schema , you perhaps also include a repository name without any placeholders first. That way, users who don't know about or care about placeholders see a plain example in which they don't have to decipher {fqdn}, and users who do care about placeholders have an example that makes sense for them.

This is great. Thank you for taking the time to make a PR.. And a test! My only request is that in the example in the `repositories` schema , you perhaps also include a repository name *without* any placeholders first. That way, users who don't know about or care about placeholders see a plain example in which they don't have to decipher `{fqdn}`, and users who do care about placeholders have an example that makes sense for them.
Poster

Ok. Maybe with

example:
  - user@backupserver:sourcehostname.borg
  - "user@backupserver:/repos/{fqdn}"
Ok. Maybe with ``` example: - user@backupserver:sourcehostname.borg - "user@backupserver:/repos/{fqdn}" ```
Owner

Yup, that'd be fine!

Yup, that'd be fine!
Poster

I force-pushed with that.

I force-pushed with that.
Owner

Awesome, thank you!

Awesome, thank you!
witten merged commit e77757f0fd into master 8 months ago
The pull request has been merged as e77757f0fd.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

This pull request currently doesn't have any dependencies.

Loading…
There is no content yet.