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

Catch java.lang.NoSuchFieldError when looking for WHEN_ENTRY_GUARD in kotlin version 2.0.1 #2857

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

paul-dingemans
Copy link
Collaborator

Description

Catch java.lang.NoSuchFieldError when looking for WHEN_ENTRY_GUARD in kotlin version 2.0.1

As this exception was not caught in the rule, and also not in the RuleExecutionContext it terminated linting/formatting of files containing a WHEN_ENTRY with an exception. In the Ktlint CLI this exception was swallowed. In ktlint-intellij-plugin the exception was recorded and became visible to users.

Closes #2856

Checklist

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

  • 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
  • KtLint format has been applied on source code itself and violations are fixed
  • PR title is short and clear (it is used as description in the release changelog)
  • 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

… kotlin version 2.0.1

As this exception was not caught in the rule, and also not in the RuleExecutionContext it terminated linting/formatting of files containing a WHEN_ENTRY with an exception. In the Ktlint CLI this exception was swallowed. In ktlint-intellij-plugin the exception was recorded and became visible to users.

Closes #2856
@paul-dingemans paul-dingemans merged commit dc4e2fb into master Nov 5, 2024
10 checks passed
@paul-dingemans paul-dingemans deleted the 2856 branch November 5, 2024 16:11
@paul-dingemans paul-dingemans added this to the 1.4.1 milestone Nov 5, 2024
orchestr7 added a commit to saveourtool/diktat that referenced this pull request Dec 7, 2024
### What's done:
- Occasionally Kotlin decided to remove several AST Nodes JetBrains/kotlin@76592dc

- There also was such kind of bug caused by Kotlin2 in ktlint pinterest/ktlint#2857

- Also removed Werror flag due to one deprecation warning which we are not able to fix

- Updated tests and Kotlin version to 2.1 in resources

- Fixed some detekt deprecation warnings

- Replaced gradle configuration with new compilerOptions API
orchestr7 added a commit to saveourtool/diktat that referenced this pull request Dec 7, 2024
### What's done:
- Occasionally Kotlin decided to remove several AST Nodes JetBrains/kotlin@76592dc

- There also was such kind of bug caused by Kotlin2 in ktlint pinterest/ktlint#2857

- Removed `-Werror` flag due to one deprecation warning which we are not able to fix

- Updated tests and Kotlin version to 2.1 in resources

- Fixed some detekt deprecation warnings

- Replaced gradle configuration with new compilerOptions API

- Removed unused methods which caused warnings

- Updated sarif4k:  it caused an error with missing 'void io.github.detekt.sarif4k.ToolComponent.<init>

Co-authored-by: akuleshov7 <andrewkuleshov7@gmail.com>
Co-authored-by: orchestr7 <bonycatbony@gmail.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ktlint-intellij-plugin stops linting/formatting file with exception
1 participant