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

Fix potential NPE in descriptor lookup #14

Merged
merged 1 commit into from
Jan 10, 2024
Merged

Fix potential NPE in descriptor lookup #14

merged 1 commit into from
Jan 10, 2024

Conversation

jckoenen
Copy link
Contributor

In our existing implementation this NPE only occurs in artificial test-data, but it might also occur for other sarif producers

In our existing implementation this NPE only occurs in artificial test-data,
but it might also occur for other sarif producers
Copy link

Qodana for JVM

It seems all right 👌

No new problems were found according to the checks applied

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/[email protected]
        with:
          upload-result: true
Dependencies licenses

Third-party software list

This page lists the third-party software dependencies used in qodana-sarif

Dependency Version Licenses
annotations 13.0 Apache-2.0
gson 2.8.9 Apache-2.0
kotlin-stdlib 1.9.21 Apache-2.0
Contact Qodana team

Contact us at [email protected]

@tiulpin tiulpin merged commit f072612 into main Jan 10, 2024
4 checks passed
@tiulpin tiulpin deleted the tools-npe branch January 10, 2024 12:00
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