Thanks to visit codestin.com
Credit goes to github.com

Skip to content

bisync: add missing rc params - fixes #7799#9225

Open
nielash wants to merge 4 commits intorclone:masterfrom
nielash:bisync-rc-params
Open

bisync: add missing rc params - fixes #7799#9225
nielash wants to merge 4 commits intorclone:masterfrom
nielash:bisync-rc-params

Conversation

@nielash
Copy link
Contributor

@nielash nielash commented Mar 3, 2026

What is the purpose of this change?

  • adds 11 missing rc params
  • makes optional params truly optional
  • adds more structured info to rc output
  • adds a go generate to auto-generate rc help docs

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

Checklist

  • I have read the contribution guidelines.
  • I have added tests for all changes in this PR if appropriate.
  • I have added documentation for the changes if appropriate.
  • All commit messages are in house style.
  • I'm done, this Pull Request is ready for review :-)

nielash added 4 commits March 2, 2026 21:24
This adds 11 previously-missing rc params for newer bisync features.

It also makes optional parameters truly optional. (Previously, callers were
required to supply every single one, even if using the default value.)
This adds a few handy bits of info, like the session name, workdir location, and
listing file paths, to the rc output.
This adds a go generate ./cmd/bisync command to autogenerate the bisync rc docs,
including the list of params.
@nielash nielash added this to the v1.74 milestone Mar 3, 2026
@nielash nielash added this to bisync Mar 3, 2026
@github-project-automation github-project-automation bot moved this to In progress in bisync Mar 3, 2026
@nielash nielash marked this pull request as ready for review March 3, 2026 03:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: In progress

Development

Successfully merging this pull request may close these issues.

1 participant