Fix bug in postgraphile CLI parser for no --watch
#1867
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
CLI flags override the config; so if you have
watch: false
in the config but run the CLI withpostgraphile --watch
then watch mode should be enabled. However because we haddefault: false
in the config for the argument parser, we were overridingwatch: true
in the config withwatch: false
from the CLI.Removing this default resolves the issue.
Issue raised by @TimoStolz via Discord: (sponsors only channel): https://discord.com/channels/489127045289476126/953710447087980654/1176875333560062145
Performance impact
None.
Security impact
None.
Checklist
yarn lint:fix
passes.I've added tests for the new feature, andyarn test
passes.I have detailed the new feature in the relevant documentation.RELEASE_NOTES.md
file (if one exists).