spmallette commented on code in PR #3289:
URL: https://github.com/apache/tinkerpop/pull/3289#discussion_r2670488336
##########
gremlint/src/formatQuery/__tests__/dotsAfterLineBreaks.test.ts:
##########
@@ -51,11 +50,10 @@ test('If dots are configured to be placed after line
breaks, make sure they are
).toBe(`g.V()
.hasLabel('person')
.group()
- .by(
- values('name', 'age').fold())
+ .by(values('name', 'age')
+ .fold())
.unfold()
- .filter(
- select(values)
- .count(local)
- .is(gt(1)))`);
+ .filter(select(values)
+ .count(local)
Review Comment:
i don't like the whole dots starting a line at all in any event - almost
wanted to remove the feature and get rid of the complexity. i guess there
could be special rules in this case to further indent, but i'm not sure what
this should be for Gremlin exactly.. 🤷
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
