Ba/correct readme config required keys #339

bradleyandrick · 2024-02-29T19:29:16Z

Related Issue(s):

Proposed Changes:

Update readme for config keys showing as required to correctly reflect app run requirements
Update config lint script to handle changes to required config keys
Fix type in config lint script

PR Checklist:

config_helper/lint_config.py

Co-authored-by: Phil Varner <[email protected]>

bradleyandrick added 3 commits February 29, 2024 14:22

update readme to correctly show required config keys

fadba2b

update config lint script to handle changed required keys

3f6cb45

update changelog

a50edef

bradleyandrick requested a review from philvarner February 29, 2024 19:29

bradleyandrick added 3 commits February 29, 2024 14:38

fix bug in required keys script

1740231

resolve issue with public URL not need to be required

5a31ecc

fix required keys in lint script

933a51b

philvarner reviewed Mar 4, 2024

View reviewed changes

config_helper/lint_config.py Outdated Show resolved Hide resolved

philvarner approved these changes Mar 4, 2024

View reviewed changes

Update config_helper/lint_config.py

e9f9f01

Co-authored-by: Phil Varner <[email protected]>

bradleyandrick merged commit 60c665e into main Mar 4, 2024
3 checks passed

bradleyandrick deleted the ba/correct-readme-config-required-keys branch August 2, 2024 17:42

Provide feedback