Skip to content
New issue

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

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

Already on GitHub? # to your account

Account for separating spaces when parsing the disabled rules. #1508

Merged

Conversation

PaulWoitaschek
Copy link
Contributor

IntelliJ reformats the disabled_rules of the editor config and adds spaces between the entries.

For example:

[*]
disabled_rules = import-ordering,filename,trailing-comma

Will be reformatted to:

[*]
disabled_rules = import-ordering, filename, trailing-comma

With the whitespace included, ktlint does not pick up these rules.

This PR accounts for these white spaces and modifies the existing tests to also cover this scenario.

@shashachu
Copy link
Contributor

thanks for the contribution!

@shashachu shashachu merged commit 4c42555 into pinterest:master Jun 20, 2022
paul-dingemans added a commit to paul-dingemans/ktlint that referenced this pull request Jun 20, 2022
@paul-dingemans paul-dingemans mentioned this pull request Jun 20, 2022
5 tasks
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants