Skip to content

Remove unused symbol check.#42

Merged
gibbiemonster merged 1 commit intomainfrom
bugfix/qodana-being-dumb
May 10, 2025
Merged

Remove unused symbol check.#42
gibbiemonster merged 1 commit intomainfrom
bugfix/qodana-being-dumb

Conversation

@gibbiemonster
Copy link
Contributor

Basically all of the native functions return an error because they are "unused" on the Kotlin side, but they are public API's exposed to Lua. Make Qodana ignore them.

Basically all of the native functions return an error because they are "unused" on the Kotlin side, but they are public API's exposed to Lua. Make Qodana ignore them.
@gibbiemonster gibbiemonster requested a review from zNotChill as a code owner May 10, 2025 23:45
Copy link
Contributor

@zNotChill zNotChill left a comment

Choose a reason for hiding this comment

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

amazing change

@gibbiemonster gibbiemonster merged commit b3d91a4 into main May 10, 2025
2 checks passed
@github-actions
Copy link

Qodana Community for JVM

26 new problems were found

Inspection name Severity Problems
Unused import directive 🔶 Warning 11
Redundant call of conversion method 🔶 Warning 2
Unresolved reference in KDoc 🔶 Warning 1
Usage of redundant or deprecated syntax or deprecated symbols 🔶 Warning 1
Redundant nullable return type 🔶 Warning 1
Class member can have 'private' visibility ◽️ Notice 5
If-Null return/break/... foldable to '?:' ◽️ Notice 2
Variable declaration could be moved inside 'when' ◽️ Notice 1
Redundant empty primary constructor ◽️ Notice 1
Accessor call that can be replaced with property access syntax ◽️ Notice 1

☁️ View the detailed Qodana report

Contact Qodana team

Contact us at qodana-support@jetbrains.com

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.

2 participants

Comments