Skip to content

Vldf/qodana#2

Closed
vldF wants to merge 19 commits intomasterfrom
vldf/qodana
Closed

Vldf/qodana#2
vldF wants to merge 19 commits intomasterfrom
vldf/qodana

Conversation

@vldF
Copy link
Owner

@vldF vldF commented Jul 31, 2025

No description provided.

Danil42Russia and others added 2 commits July 30, 2025 15:50
* add stub tests

* add todo

* update kphp-template test

* use `KphpStormTestBase` as a base class for `StubTestBase`

* Add missing tests

* backport new approach to walt a stub tree
@github-actions
Copy link

github-actions bot commented Jul 31, 2025

Qodana Community for JVM

11 new problems were found

Inspection name Severity Problems
Plugin.xml validity 🔴 Failure 1
XML highlighting 🔴 Failure 1
Redundant suppression 🔶 Warning 4
Plugin.xml validity 🔶 Warning 2
Typo ◽️ Notice 3

💡 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:

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@v2025.1.1
        with:
          upload-result: true
Contact Qodana team

Contact us at qodana-support@jetbrains.com

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.

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

@vldF
Copy link
Owner Author

vldF commented Aug 8, 2025

plugin.xml анализировать умеем

image image

@vldF vldF force-pushed the vldf/qodana branch 2 times, most recently from a9f9147 to 2734db0 Compare August 8, 2025 13:05
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