-
Notifications
You must be signed in to change notification settings - Fork 346
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
Implement "config set" subcommand #1312
Conversation
Hmm, my "cargo fmt" isn't wrapping these lines but the "Check formatting" insists the long line needs to be wrapped. Any idea why? I just did rustup update.
|
We use the nightly version in CI (I don't remember if there's a good reason for that), so try |
d50384f
to
17ce84f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks.
31d4a0d
to
367199f
Compare
Uses toml_edit to support simple config edits like: jj config set --repo user.email "[email protected]"
Uses toml_edit to support simple config edits like:
jj config set --repo user.email "[email protected]"
Fixes #531.
Checklist
If applicable:
CHANGELOG.md