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.

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

Indentation rule rewrite #1682

Merged
merged 103 commits into from
Oct 25, 2022

Conversation

paul-dingemans
Copy link
Collaborator

@paul-dingemans paul-dingemans commented Oct 24, 2022

Description

The indent rule has been rewritten from scratch. Solving problems in the old algorithm was very difficult. With the new algorithm this becomes a lot easier. Although the new implementation of the rule has been compared against several open source projects containing over 400,000 lines of code, it is still likely that new issues will be discovered. Please report your indentation issues so that these can be fixed as well.

Closes #1321
Closes #1200
Closes #1562
Closes #1563
Closes #1639

Checklist

  • PR description added
  • tests are added
  • KtLint has been applied on source code itself and violations are fixed
  • documentation is updated
  • CHANGELOG.md is updated

In case of adding a new rule:

…id conflict with IntelliJ IDEA default formatting

Closes 1682
Closes 1321
Closes 1200
Closes 1562
Closes 1563
Closes 1639
@paul-dingemans paul-dingemans merged commit 750f214 into pinterest:master Oct 25, 2022
@paul-dingemans paul-dingemans added this to the 0.48.0 milestone Oct 26, 2022
@paul-dingemans paul-dingemans deleted the indentation-rule-rewrite branch October 26, 2022 17:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment