Skip to content

Conversation

@breedloj
Copy link
Contributor

@breedloj breedloj commented Dec 2, 2025

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)

Description

Removed additional plugin ID references to Core.

Checklist

  • My code follows the code style of this project
  • I have added tests to cover my changes
  • A short description of the change has been added to the CHANGELOG if the change is customer-facing in the IDE.
  • I have added metrics for my changes (if required)

License

I confirm that my contribution is made under the terms of the Apache 2.0 license.

@breedloj breedloj requested a review from a team as a code owner December 2, 2025 22:53
@github-actions
Copy link

github-actions bot commented Dec 2, 2025

Qodana Community for JVM

It seems all right 👌

No new problems were found according to the checks applied

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

@breedloj breedloj requested a review from a team as a code owner December 24, 2025 00:15
@github-actions
Copy link

github-actions bot commented Dec 24, 2025

Qodana for JVM

358 new problems were found

Inspection name Severity Problems
Usage of API marked for removal 🔴 Failure 22
Unstable API Usage 🔴 Failure 2
Unused symbol 🔶 Warning 114
Unreachable code 🔶 Warning 32
Extension class should be final and non-public 🔶 Warning 22
Unused import directive 🔶 Warning 19
Redundant 'Companion' reference 🔶 Warning 18
Unstable API Usage 🔶 Warning 13
Incorrect string capitalization 🔶 Warning 10
'Enum.values()' is recommended to be replaced by 'Enum.entries' since 1.9 🔶 Warning 7
Redundant qualifier name 🔶 Warning 5
Companion object in extensions 🔶 Warning 4
Java default methods are not overridden by delegation 🔶 Warning 4
Constructor parameter is never used as a property 🔶 Warning 3
Redundant nullable return type 🔶 Warning 3
Redundant visibility modifier 🔶 Warning 3
Component/Action not registered 🔶 Warning 2
Method can only be overridden 🔶 Warning 2
Redundant receiver-based 'let' call 🔶 Warning 2
Possibly blocking call in non-blocking context 🔶 Warning 1
Private data class constructor is exposed via the 'copy' method 🔶 Warning 1
Nullable type argument in Java collection that doesn't support nulls 🔶 Warning 1
Constant conditions 🔶 Warning 1
Redundant modality modifier 🔶 Warning 1
Redundant SAM constructor 🔶 Warning 1
Redundant upper bound 'Any?' 🔶 Warning 1
Use Darcula aware JBColor 🔶 Warning 1
Redundant call on collection type 🔶 Warning 1
Function or property has platform type ◽️ Notice 17
Accessor call that can be replaced with property access syntax ◽️ Notice 8
Redundant interpolation prefix ◽️ Notice 5
Java Map.forEach method call should be replaced with Kotlin's forEach ◽️ Notice 4
Redundant 'if' statement ◽️ Notice 4
Explicit 'get' or 'set' call ◽️ Notice 3
Duplicated code fragment ◽️ Notice 2
If-Then foldable to '?:' ◽️ Notice 2
Redundant overriding method ◽️ Notice 2
Return or assignment can be lifted out ◽️ Notice 2
Non-distinguishable logging calls ◽️ Notice 2
Nested lambda has shadowed implicit parameter ◽️ Notice 2
Private property naming convention ◽️ Notice 2
Boolean expression can be simplified ◽️ Notice 2
Two comparisons should be converted to a range check ◽️ Notice 1
Redundant 'asSequence' call ◽️ Notice 1
'map.put()' can be converted to assignment ◽️ Notice 1
Size check can be replaced with 'isNotEmpty()' ◽️ Notice 1
Call chain on collection type can be simplified ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked
☁️ View the detailed Qodana report

Contact Qodana team

Contact us at [email protected]

}

override fun <T> getResourceIfPresent(resource: Resource<T>, region: AwsRegion, credentialProvider: ToolkitCredentialsProvider, useStale: Boolean): T? =
when (resource) {

Check notice

Code scanning / QDJVM

Duplicated code fragment Note

Duplicated code
}

override fun invalidateAccessToken(ssoUrl: String) {
LOG.info { "invalidateAccessToken for $ssoUrl" }

Check notice

Code scanning / QDJVM

Duplicated code fragment Note

Duplicated code
Copy link

@github-advanced-security github-advanced-security bot left a comment

Choose a reason for hiding this comment

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

QDJVM found more than 20 potential problems in the proposed changes. Check the Files changed tab for more details.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants