Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Integrate ktlint into Gradle build #43

Closed
raganhan opened this issue May 17, 2019 · 1 comment · Fixed by #542
Closed

Integrate ktlint into Gradle build #43

raganhan opened this issue May 17, 2019 · 1 comment · Fixed by #542
Labels
build good first issue Good for newcomers S Small sized task

Comments

@raganhan
Copy link
Contributor

In order to facilitate consistency in code formatting we should see investigate if ktlint can reformat our code as part of (every?) build.

This will also require re-evaluating CODE-STYLE.md and making changes as necessary.
@raganhan raganhan added the build label May 17, 2019
@therapon therapon added the S Small sized task label Feb 13, 2020
@therapon
Copy link
Contributor

We added detekt but we need to decide the rules/checks first and configure accordingly.

@dlurton dlurton added the good first issue Good for newcomers label Jun 26, 2020
@alancai98 alancai98 mentioned this issue Feb 9, 2022
Merged
@alancai98 alancai98 mentioned this issue Mar 3, 2022
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
build good first issue Good for newcomers S Small sized task
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adds ktlint to gradle build partiql/partiql-lang-kotlin
4 participants
@raganhan @therapon @dlurton @alancai98