Adds 'lading config-check' that checks whether or not a given config is valid#1396
Merged
scottopell merged 2 commits intomainfrom Jul 15, 2025
Merged
Adds 'lading config-check' that checks whether or not a given config is valid#1396scottopell merged 2 commits intomainfrom
scottopell merged 2 commits intomainfrom
Conversation
Contributor
Author
This was referenced Jun 17, 2025
scottopell
force-pushed
the
sopell/add-run-subcommand
branch
from
ab0361c to
beebeae
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
17712d6 to
9d8e90a
Compare
scottopell
force-pushed
the
sopell/add-run-subcommand
branch
from
beebeae to
a290872
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
9d8e90a to
4b0ebb9
Compare
scottopell
force-pushed
the
sopell/add-run-subcommand
branch
from
a290872 to
189d9d9
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
4b0ebb9 to
01cac6f
Compare
scottopell
force-pushed
the
sopell/add-run-subcommand
branch
from
189d9d9 to
9e9fc0f
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
01cac6f to
144a4ec
Compare
scottopell
changed the title
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
2 times, most recently
from
6b50689 to
7ee5588
Compare
blt
approved these changes
Jun 18, 2025
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
7ee5588 to
b496152
Compare
scottopell
force-pushed
the
sopell/add-run-subcommand
branch
from
a858767 to
68203bd
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
b496152 to
1546a52
Compare
scottopell
force-pushed
the
sopell/add-run-subcommand
branch
3 times, most recently
from
6717a4c to
9c87dcf
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
1546a52 to
d91a955
Compare
scottopell
changed the base branch from
sopell/add-run-subcommand
to
graphite-base/1396
…defaults/env/cli options as 'run' and exits 0 or 1 based on whether the config is valid
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
d91a955 to
9727b1e
Compare
scottopell
force-pushed
the
graphite-base/1396
branch
from
9c87dcf to
79a32d3
Compare
scottopell
force-pushed
the
sopell/add-config-check-subcommand
branch
from
9727b1e to
1ddff18
Compare
Contributor
Author
Merge activity
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
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.
What does this PR do?
Adds a new sub-command
lading config-checkthat follows the same "config" behavior and exits 0 or 1 based on whether the config was valid or not.Motivation
Allow a faster failure feedback loop.
Related issues
Additional Notes