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.

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

config: move resolved_config_values() to free function #4967

Merged
merged 4 commits into from
Nov 26, 2024

Conversation

yuja
Copy link
Contributor

@yuja yuja commented Nov 25, 2024

Checklist

If applicable:

  • I have updated CHANGELOG.md
  • I have updated the documentation (README.md, docs/, demos/)
  • I have updated the config schema (cli/src/config-schema.json)
  • I have added tests to cover my changes

yuja added 4 commits November 25, 2024 18:15
There's only one caller, and it will become accessible through
command.settings().
This will help replace cli LayeredConfigs with new StackedConfig. I originally
considered moving this function to jj-lib, but the current API is very specific
to the CLI use case, and wouldn't be reused for building a merged sub table. We
might instead want to extract some bits as a library function.
The field name "path" was confusing because we'll probably add a source file
path to AnnotatedValue.
@yuja yuja merged commit e029e8d into jj-vcs:main Nov 26, 2024
18 checks passed
@yuja yuja deleted the push-kolwxmowqrwl branch November 26, 2024 09:17
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.

2 participants