GNU bug report logs -
#68054
[PATCH] Add tree-sitter indent rule for lexical decls in js/typescript
Previous Next
Reported by: Noah Peart <noah.v.peart <at> gmail.com>
Date: Wed, 27 Dec 2023 06:22:01 UTC
Severity: wishlist
Tags: patch
Done: Stefan Kangas <stefankangas <at> gmail.com>
Bug is archived. No further changes may be made.
Full log
Message #31 received at 68054 <at> debbugs.gnu.org (full text, mbox):
> On Apr 17, 2024, at 1:21 PM, Noah Peart <noah.v.peart <at> gmail.com> wrote:
>
> Sorry, I forgot about this. I've just added a rule to align the variable_declarators
> in let, var, and const declarations, but I need some feedback about the indentation
> for values in the variable_declarators following dangling '='.
Thanks!
>
> For example, which of the following would be preferable?
>
> 1) indent the dangling values with respect to start of the declaration
>
> const a =
> (x: string): string => {
> return x + x;
> },
> bbb =
> {
> "x": 0
> },
> cccc =
> 1,
> ddddd = 0;
>
> 2) indent them with respect to the start of the variable_declarator
>
> const a =
> (x: string): string => {
> return x + x;
> },
> bbb =
> {
> "x": 0
> },
> cccc =
> 1,
> ddddd = 0;
>
> 3) align with the variable declarators (this is the same as js-mode)
>
> const a =
> (x: string): string => {
> return x + x;
> },
> bbb =
> {
> "x": 0
> },
> cccc =
> 1,
> ddddd = 0;
>
> I've attached a patch with with the rules for the 3 options here.
I don’t really know what’s the convention, if there is one. Maybe Dmitry has better idea. Personally I like option 1.
Yuan
This bug report was last modified 191 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.