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

Update com.pinterest.ktlint to v0.45.0 #4641

Closed
wants to merge 1 commit into from

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 18, 2022

WhiteSource Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
com.pinterest.ktlint:ktlint-ruleset-experimental 0.43.2 -> 0.45.0 age adoption passing confidence
com.pinterest.ktlint:ktlint-ruleset-standard 0.43.2 -> 0.45.0 age adoption passing confidence
com.pinterest.ktlint:ktlint-core 0.43.2 -> 0.45.0 age adoption passing confidence

Release Notes

pinterest/ktlint

v0.45.0

Compare Source

API Changes & RuleSet providers

If you are not an API user nor a RuleSet provider, then you can safely skip this section. Otherwise, please read below carefully and upgrade your usage of ktlint. In this and coming releases, we are changing and adapting important parts of our API in order to increase maintainability and flexibility for future changes. Please avoid skipping a releases as that will make it harder to migrate.

Retrieving ".editorconfig" property value

This section is applicable when providing rules that depend on one or more values of ".editorconfig" properties. Property values should no longer be retrieved via EditConfig or directly via userData[EDITOR_CONFIG_USER_DATA_KEY]. Property values should now only be retrieved using method ASTNode.getEditorConfigValue(editorConfigProperty) of interface UsesEditorConfigProperties which is provided in this release. Starting from next release after the current release, the EditConfig and/or userData[EDITOR_CONFIG_USER_DATA_KEY] may be removed without further notice which will break your API or rule. To prevent disruption of your end user, you should migrate a.s.a.p.

Added
  • Add experimental rule for unexpected spaces in a type reference before a function identifier (function-type-reference-spacing) (#​1341)
  • Add experimental rule for incorrect spacing after a type parameter list (type-parameter-list-spacing) (#​1366)
  • Add experimental rule to detect discouraged comment locations (discouraged-comment-location) (#​1365)
  • Add rule to check spacing after fun keyword (fun-keyword-spacing) (#​1362)
  • Add experimental rules for unnecessary spacing between modifiers in and after the last modifier in a modifier list (#​1361)
  • New experimental rule for aligning the initial stars in a block comment when present (experimental:block-comment-initial-star-alignment (#​297)
  • Respect .editorconfig property ij_kotlin_packages_to_use_import_on_demand (no-wildcard-imports) (#​1272)
  • Add new experimental rules for wrapping of block comment (comment-wrapping) (#​1403)
  • Add new experimental rules for wrapping of KDoc comment (kdoc-wrapping) (#​1403)
  • Add experimental rule for incorrect spacing after a type parameter list (type-parameter-list-spacing) (#​1366)
  • Expand check task to run tests on JDK 17 - "testOnJdk17"
Fixed
  • Fix lint message to "Unnecessary long whitespace" (no-multi-spaces) (#​1394)
  • Do not remove trailing comma after a parameter of type array in an annotation (experimental:trailing-comma) (#​1379)
  • Do not delete blank lines in KDoc (no-trailing-spaces) (#​1376)
  • Do not indent raw string literals that are not followed by either trimIndent() or trimMargin() (indent) (#​1375)
  • Revert remove unnecessary wildcard imports as introduced in Ktlint 0.43.0 (no-unused-imports) (#​1277), (#​1393), (#​1256)
  • (Possibly) resolve memory leak (#​1216)
  • Initialize loglevel in Main class after parsing the CLI parameters (#​1412)
Changed
  • Print the rule id always in the PlainReporter (#​1121)
  • All wrapping logic is moved from the indent rule to the new rule wrapping (as part of the standard ruleset). In case you currently have disabled the indent rule, you may want to reconsider whether this is still necessary or that you also want to disable the new wrapping rule to keep the status quo. Both rules can be run independent of each other. (#​835)
Removed

v0.44.0

Compare Source

Please welcome paul-dingemans as an official maintainer of ktlint!

Added
  • Use Gradle JVM toolchain with language version 8 to compile the project
  • Basic tests for CLI (#​540)
  • Add experimental rule for unnecessary parentheses in function call followed by lambda (#​1068)
Fixed
  • Fix indentation of function literal (#​1247)
  • Fix false positive in rule spacing-between-declarations-with-annotations (#​1281)
  • Do not remove imports for same class when different alias is used (#​1243)
  • Fix NoSuchElementException for property accessor (trailing-comma) (#​1280)
  • Fix ClassCastException using ktlintFormat on class with KDoc (no-trailing-spaces) (#​1270)
  • Do not remove trailing comma in annotation (#​1297)
  • Do not remove import which is used as markdown link in KDoc only (no-unused-imports) (#​1282)
  • Fix indentation of secondary constructor (indent) (#​1222)
  • Custom gradle tasks with custom ruleset results in warning (#​1269)
  • Fix alignment of arrow when trailing comma is missing in when entry (trailing-comma) (#​1312)
  • Fix indent of delegated super type entry (indent) (#​1210)
  • Improve indentation of closing quotes of a multiline raw string literal (indent) (#​1262)
  • Trailing space should not lead to delete of indent of next line (no-trailing-spaces) (#​1334)
  • Force a single line function type inside a nullable type to a separate line when the max line length is exceeded (parameter-list-wrapping) (#​1255)
  • A single line function with a parameter having a lambda as default argument does not throw error (indent) (#​1330)
  • Fix executable jar on Java 16+ (#​1195)
  • Fix false positive unused import after autocorrecting a trailing comma (#​1367)
  • Fix false positive indentation (parameter-list-wrapping, argument-list-wrapping) (#​897, #​1045, #​1119, #​1255, #​1267, #​1319, #​1320, #​1337
  • Force a single line function type inside a nullable type to a separate line when the max line length is exceeded (parameter-list-wrapping) (#​1255)
Changed
  • Update Kotlin version to 1.6.0 release
  • Add separate tasks to run tests on JDK 11 - "testOnJdk11"
  • Update Dokka to 1.6.0 release
  • Apply ktlint experimental rules on the ktlint code base itself.
  • Update shadow plugin to 7.1.1 release
  • Add Kotlin-logging backed by logback as logging framework (#​589)
  • Update Gradle to 7.4 version

Configuration

📅 Schedule: At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot added the dependencies Pull requests that update a dependency file label Mar 18, 2022
@chao2zhang
Copy link
Member

Let's merge #4630 first

@chao2zhang chao2zhang closed this Mar 18, 2022
@chao2zhang chao2zhang deleted the renovate/com.pinterest.ktlint branch March 18, 2022 21:59
@renovate
Copy link
Contributor Author

renovate bot commented Mar 18, 2022

Renovate Ignore Notification

As this PR has been closed unmerged, Renovate will now ignore this update (0.45.0). You will still receive a PR once a newer version is released, so if you wish to permanently ignore this dependency, please add it to the ignoreDeps array of your renovate config.

If this PR was closed by mistake or you changed your mind, you can simply rename this PR and you will soon get a fresh replacement PR opened.

Copy link
Contributor

@Goooler Goooler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants