Skip to content
New issue

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

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

config: allow empty configs, but prevent bad configs #4394

Merged
merged 2 commits into from Aug 15, 2023
Merged

config: allow empty configs, but prevent bad configs #4394

merged 2 commits into from Aug 15, 2023

Commits on Aug 14, 2023

  1. config: allow empty configs, but prevent bad configs 

    - Adds reporting the line with the bad key position
  that makes the config invalid.

- Fixes a few tests with trailing braces which were
  being handled as keys and ignored before.

Signed-off-by: Waldemar Quevedo <wally@nats.io>
    @wallyqs
    wallyqs committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    412dee6 View commit details
    Browse the repository at this point in the history

  2. config: parsed empty config only show warnings 

    Signed-off-by: Waldemar Quevedo <wally@nats.io>
    @wallyqs
    wallyqs committed Aug 14, 2023
    Configuration menu
    Copy the full SHA
    3a20f66 View commit details
    Browse the repository at this point in the history