neo4j-contrib / sql2cypher

Experimental SQL to Cypher Transpiler using jooq and cypher-dsl
Apache License 2.0
26 stars 1 forks source link

Settle on a format. #12

Closed michael-simons closed 1 year ago

michael-simons commented 1 year ago

I guess we all have different ideas about source code formatting (I'm team tab btw), but I have no ambition to discuss this.

Can we pick one:

I'm happy to set any of those up.

lukaseder commented 1 year ago

I don't have an opinion, just used IntelliJ's defaults (and I don't use IntelliJ often).

So please, set up any automation of your preference, probably ideally the same one as the one from cypher-dsl? Once I see more of your code, I'll also try to adapt to other coding styles that you prefer.

jexp commented 1 year ago

I don't have an opinion either, (except for not using the curly brace on new line format :) Given my ADHD I also usually forget to/don't pay attention to things like formatting, so please bear with me. But happy to have a pre-commit hook for formatting.

So pick whichever you like @michael-simons and set it up and we'll follow it.

I wouldn't want to use something from Palantir though.

michael-simons commented 1 year ago

Done. See reasoning in the commit message.