Skip to content

190 update default config file docs and values#193

Merged
thecaffiend merged 11 commits into
mainfrom
190-update-default-config-file-docs-and-values
Dec 11, 2024
Merged

190 update default config file docs and values#193
thecaffiend merged 11 commits into
mainfrom
190-update-default-config-file-docs-and-values

Conversation

@thecaffiend
Copy link
Copy Markdown
Member

@thecaffiend thecaffiend commented Dec 11, 2024

TO TEST

Review. There is one code change to force a failure on a missing config key, but otherwise should be nothing but doc changes.

There was a deployment associated with this branch with a single change. The description of ccd-pvsl-dap-api-lsdaps-lmbdfn changed due to the description being moved to the config file to provide an example (previously it was defaulted and not in the config at all).

@thecaffiend thecaffiend linked an issue Dec 11, 2024 that may be closed by this pull request
@thecaffiend thecaffiend requested a review from mehalter December 11, 2024 16:38
Copy link
Copy Markdown
Member

@mehalter mehalter left a comment

Choose a reason for hiding this comment

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

Looks great! :shipit:

@thecaffiend thecaffiend merged commit 77a4be1 into main Dec 11, 2024
@thecaffiend thecaffiend deleted the 190-update-default-config-file-docs-and-values branch December 11, 2024 19:27
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.

Update default config file docs and values

2 participants