Skip to content

[IMPROVED] First stab at documenting varz and related structures#7080

Merged
ripienaar merged 2 commits intonats-io:mainfrom
ripienaar:varz_docs
Jul 21, 2025
Merged

[IMPROVED] First stab at documenting varz and related structures#7080
ripienaar merged 2 commits intonats-io:mainfrom
ripienaar:varz_docs

Conversation

@ripienaar
Copy link
Copy Markdown
Contributor

I'd like to produce JSON schemas for these variables so this is the first step toward that

Signed-off-by: R.I.Pienaar rip@devco.net

Signed-off-by: R.I.Pienaar <rip@devco.net>
@ripienaar ripienaar requested a review from a team as a code owner July 18, 2025 10:42
Copy link
Copy Markdown
Member

@neilalexander neilalexander left a comment

Choose a reason for hiding this comment

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

Few things I spotted.

Signed-off-by: R.I.Pienaar <rip@devco.net>
Copy link
Copy Markdown
Member

@MauriceVanVeen MauriceVanVeen left a comment

Choose a reason for hiding this comment

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

LGTM

@ripienaar ripienaar merged commit a6552bf into nats-io:main Jul 21, 2025
43 of 45 checks passed
@ripienaar ripienaar deleted the varz_docs branch July 21, 2025 09:09
@wallyqs wallyqs changed the title First stab at documenting varz and related structures [IMPROVED] First stab at documenting varz and related structures Jul 24, 2025
@alexbozhenko
Copy link
Copy Markdown
Member

Yay!
So now we can browse docs for those fields here:
https://pkg.go.dev/github.com/nats-io/nats-server/v2@main/server#JetStreamConfig

neilalexander added a commit that referenced this pull request Jul 25, 2025
Includes the following:

- #7031
- #7033
- #7034
- #7035
- #7036
- #7040
- #7043
- #7045
- #7047
- #7046
- #7050
- #7051
- #7052
- #7053
- #7061
- #7063
- #7064
- #7065
- #7066
- #7070
- #7072
- #7080
- #7026
- #6728
- #7074
- #7089
- #7095
- #7087
- #7094
- #7096
- #7099

Signed-off-by: Neil Twigg <neil@nats.io>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants