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 has Changed?
calva.fmt.configPath === 'CLOJURE-LSP'
where Calva will pick the cljfmt config from clojure-lsp.:align-associative? true
in configDoing this I also cleaned up some of the code around how cljfmt config is provided to Calva's formatter. Hopefully this is a tad less confusing now.
Resolves #1545
Resolves #1535
NB: Latest clojure-lsp does not have the necessary support for Calva to pick up config there. To test it you'll need to use a clojure-lsp build from
master
.My Calva PR Checklist
I have:
dev
branch. (Or have specific reasons to target some other branch.)published
. (Sorry for the nagging.)[Unreleased]
entry inCHANGELOG.md
, linking the issue(s) that the PR is addressing.ci/circleci: build
test.npm run prettier-format
)npm run eslint
).Ping @PEZ, @bpringe