Simplify default properties loading - Githubissues

pinterest / ktlint

An anti-bikeshedding Kotlin linter with built-in formatter

https://pinterest.github.io/ktlint/

MIT License

6.06k stars 504 forks source link

Simplify default properties loading #2679

Closed Goooler closed 4 weeks ago

Goooler commented 1 month ago

Description

The file path will be checked in com.pinterest.ktlint.rule.engine.internal.EditorConfigDefaultsLoader.load, no need to check nullability again.

Checklist

Before submitting the PR, please check following (checks which are not relevant may be ignored):

[x] Commit message are well written. In addition to a short title, the commit message also explain why a change is made.
[ ] At least one commit message contains a reference Closes #<xxx> or Fixes #<xxx> (replace<xxx> with issue number)
[ ] Tests are added
[x] KtLint format has been applied on source code itself and violations are fixed
[x] PR title is short and clear (it is used as description in the release changelog)
[x] PR description added (background information)

Documentation is updated. See difference between snapshot and release documentation

[ ] Snapshot documentation in case documentation is to be released together with a code change
[ ] Release documentation in case documentation is related to a released version of ktlint and has to be published as soon as the change is merged to master