Skip to content

docs: improve wild(*) section#1228

Merged
joergsteffens merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/docs-improve-wild-section
Aug 22, 2022
Merged

docs: improve wild(*) section#1228
joergsteffens merged 2 commits intobareos:masterfrom
bruno-at-bareos:dev/bruno/master/docs-improve-wild-section

Conversation

@bruno-at-bareos
Copy link
Contributor

@bruno-at-bareos bruno-at-bareos commented Aug 16, 2022

This PR improve the Wild(*) section of the documentation

Please check

  • Short description and the purpose of this PR is present above this paragraph
  • Your name is present in the AUTHORS file (optional)

If you have any questions or problems, please give a comment in the PR.

Helpful documentation and best practices

Checklist for the reviewer of the PR (will be processed by the Bareos team)

General
  • PR name is meaningful
  • Purpose of the PR is understood
  • Separate commit for this PR in the CHANGELOG.md, PR number referenced is same
  • Commit descriptions are understandable and well formatted
  • [ ] If backport: add original PR number and target branch at top of this file: Backport of PR#000 to bareos-2x
Source code quality
  • [ ] Source code changes are understandable
  • [ ] Variable and function names are meaningful
  • [ ] Code comments are correct (logically and spelling)
  • Required documentation changes are present and part of the PR
  • bareos-check-sources --since-merge does not report any problems
  • git status should not report modifications in the source tree after building and testing

@bruno-at-bareos bruno-at-bareos added documentation This PR changes only documentation requires backport to 21 labels Aug 16, 2022
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/docs-improve-wild-section branch from 0eac521 to 6eea323 Compare August 16, 2022 12:19
@joergsteffens joergsteffens self-requested a review August 17, 2022 15:22
Copy link
Member

@joergsteffens joergsteffens left a comment

Choose a reason for hiding this comment

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

Nice. I've a few suggestions, but other than that, I approve.

@bruno-at-bareos
Copy link
Contributor Author

Thanks for the review and suggestions, I've now applied them all.

@joergsteffens
Copy link
Member

Can you pleas resolve the CHANGELOG conflict?

@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/docs-improve-wild-section branch from ae55f89 to 931b638 Compare August 17, 2022 16:11
@bruno-at-bareos
Copy link
Contributor Author

done

@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/docs-improve-wild-section branch from 931b638 to 81f27ae Compare August 22, 2022 09:35
@joergsteffens
Copy link
Member

Thanks for applying the changes. Would you mind to squash the 3 commits together?

bruno-at-bareos and others added 2 commits August 22, 2022 14:27
- add a link to each section to FileSet Examples where the usage is explained.
- add :bcommand:`estimate job=job-name listing` when estimate is shown
- fix typos, and spacing.

Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
Co-authored-by: Jörg Steffens <joergsteffens@users.noreply.github.com>
Signed-off-by: Bruno Friedmann <bruno.friedmann@bareos.com>
@bruno-at-bareos bruno-at-bareos force-pushed the dev/bruno/master/docs-improve-wild-section branch from 867589a to a059484 Compare August 22, 2022 12:27
@bruno-at-bareos
Copy link
Contributor Author

done

@joergsteffens joergsteffens merged commit cf639da into bareos:master Aug 22, 2022
@bruno-at-bareos bruno-at-bareos deleted the dev/bruno/master/docs-improve-wild-section branch August 23, 2022 08:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation This PR changes only documentation requires backport to 21

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants