Skip to content

refactoring.#10

Open
bbortt wants to merge 8 commits into
mainfrom
feat/postfinance-code
Open

refactoring.#10
bbortt wants to merge 8 commits into
mainfrom
feat/postfinance-code

Conversation

@bbortt

@bbortt bbortt commented Sep 16, 2025

Copy link
Copy Markdown
Member

No description provided.

@bbortt bbortt self-assigned this Sep 16, 2025
@github-actions

Copy link
Copy Markdown

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

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

@bbortt
bbortt force-pushed the feat/postfinance-code branch from 0acbf70 to 972580d Compare September 17, 2025 05:24
@github-actions

github-actions Bot commented Sep 17, 2025

Copy link
Copy Markdown

Qodana Community for JVM

7 new problems were found

Inspection name Severity Problems
Class is exposed outside of its visibility scope 🔶 Warning 3
@NotNull/@Nullable problems 🔶 Warning 3
Nullability and data flow problems 🔶 Warning 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:

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

Contact us at qodana-support@jetbrains.com

since migration to plugin api v2 start button has been disabled.
that has now been fixed, with updates when reading the virtual files from context.
previously, a virtual files array existed, but now the key is `PlatformCoreDataKeys.SELECTED_ITEMS`.
E.g. `AllIcons.Execute` was missing, instead `AllIcons.Actions.Execute` must be used.
I initially misread the class hierarchy.
by appending the actual test name, the run configuration is uniquely identifiable.
@bbortt
bbortt force-pushed the feat/postfinance-code branch from 820f00c to 3a22674 Compare September 22, 2025 13:41
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.

1 participant