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.

Sign up for GitHub

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

Jump to bottom

Override to BGT, fix issue 814 #815

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Override to BGT, fix issue 814 #815

wants to merge 2 commits into from

Conversation

cies
Copy link
Contributor

@cies cies commented Apr 15, 2024

Fixes #814

@cies cies self-assigned this Apr 15, 2024
@github-actions GitHub Actions
Copy link

github-actions bot commented Apr 15, 2024 •
edited

Qodana Community for JVM

189 new problems were found

Inspection name Severity Problems
Usage of API marked for removal 🔴 Failure 1
Unstable API Usage 🔴 Failure 1
Unused symbol 🔶 Warning 46
Incorrect string capitalization 🔶 Warning 17
Constant conditions 🔶 Warning 13
Redundant nullable return type 🔶 Warning 9
Unresolved reference in KDoc 🔶 Warning 7
Unused import directive 🔶 Warning 5
Unstable API Usage 🔶 Warning 4
Inconsistent comment for Java parameter 🔶 Warning 3
Redundant qualifier name 🔶 Warning 3
Usage of redundant or deprecated syntax or deprecated symbols 🔶 Warning 1
Multiple operators with different precedence 🔶 Warning 1
Boolean literal argument without parameter name ◽️ Notice 18
Class member can have 'private' visibility ◽️ Notice 12
Convert 'object' to 'data object' ◽️ Notice 6
Accessor call that can be replaced with property access syntax ◽️ Notice 6
Might be 'const' ◽️ Notice 5
Nested lambda has shadowed implicit parameter ◽️ Notice 5
Unnecessary type argument ◽️ Notice 4
Function or property has platform type ◽️ Notice 3
Private property naming convention ◽️ Notice 3
'when' that can be simplified by introducing an argument ◽️ Notice 2
Constant conditions ◽️ Notice 2
Return or assignment can be lifted out ◽️ Notice 2
Cascade 'if' can be replaced with 'when' ◽️ Notice 1
@Deprecated annotation without 'replaceWith' argument ◽️ Notice 1
Lambda argument inside parentheses ◽️ Notice 1
Variable declaration could be moved inside 'when' ◽️ Notice 1
Object literal can be converted to lambda ◽️ Notice 1
Redundant 'constructor' keyword ◽️ Notice 1
Redundant 'if' statement ◽️ Notice 1
Range can be converted to indices or iteration ◽️ Notice 1
Call chain on collection type can be simplified ◽️ Notice 1
Augmented assignment creates a new collection under the hood ◽️ Notice 1

💡 Qodana analysis was run in the pull request mode: only the changed files were checked

View the detailed Qodana report

To be able to view the detailed Qodana report, you can either:

  1. Register at Qodana Cloud and configure the action
  2. Use GitHub Code Scanning with Qodana
  3. Host Qodana report at GitHub Pages
  4. Inspect and use qodana.sarif.json (see the Qodana SARIF format for details)

To get *.log files or any other Qodana artifacts, run the action with upload-result option set to true,
so that the action will upload the files as the job artifacts:

      - name: 'Qodana Scan'
        uses: JetBrains/qodana-action@v2023.3.1
        with:
          upload-result: true
Contact Qodana team

Contact us at qodana-support@jetbrains.com

@cies cies added this to the 5.0.1 milestone Apr 17, 2024
@cies
Copy link
Contributor Author

cies commented Apr 23, 2024

This, in its current shape, introduces some test failures.

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

@cies cies

Labels
None yet
Projects
None yet
Milestone
5.0.1
Development

Successfully merging this pull request may close these issues.

com.intellij.diagnostic.PluginException for deprecated ActionUpdateThread.OLD_EDT
1 participant
@cies