BetterThanTomorrow / BetterThanTomorrow/calva

new indent engine - indent 1 character too many

Open
#809 1 comment 0 reactions 0 assignees View on GitHub
bug formatting
Dominant language
TypeScript
Stars
2.1k
Forks
249
Avg merge
6h 24m
Merged PRs (30d)
9

Description

When using the `Fmt: New Indent Engine` if I have the following configuration within my custom `cljfmt` configuration indents are indented 1 [extra space](https://github.com/BetterThanTomorrow/calva/blob/f72ebe1e2d497e8505fe65d66e24f647401cd2df/src/cursor-doc/indent.ts#L135).

`{:indents ^:replace {#".*" [[:inner 0]]}}`

Contributor guide

Open the contributing guide

Research direction

Start in src/cursor-doc/indent.ts around line 135 and reproduce the issue with the shown cljfmt configuration and the New Indent Engine. Trace how the [:inner 0] indent is calculated, then verify that the same configuration no longer adds one extra space.

Written by the indexing model from the issue text.

Assessment

Tech stack
clojure, typescript
Domain
tooling
Issue type
Bug
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.