You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@struts.apache.org by "lukaszlenart (via GitHub)" <gi...@apache.org> on 2026/02/24 16:37:51 UTC
[PR] fix: resolve 22 Qodana warnings from CI analysis [struts-intellij-plugin]
lukaszlenart opened a new pull request, #52:
URL: https://github.com/apache/struts-intellij-plugin/pull/52
## Summary
- Exclude `src/main/gen` from Qodana scanning (eliminates 9 generated code warnings from JFlex/Grammar Kit)
- Replace redundant `Collection.addAll()` with constructor initialization in `ExceptionMappingResultResolveConverter`
- Fix 7 incorrect string capitalization warnings across bundle properties and source files
- Remove unnecessary null check before `equals()` in `StrutsFileSet.hasFile()`
- Replace string concatenation in loop with `StringBuilder` in `HardcodedActionUrlInspection`
- Add `@SuppressWarnings("UnstableApiUsage")` for 2 usages with no public replacements
- Migrate `Struts2GraphComponent` from deprecated `DataProvider` to `UiDataProvider`
## Test plan
- [x] `./gradlew build` passes
- [x] `./gradlew test -x rat` passes with no regressions
- [ ] Verify Qodana CI reports 0 new warnings on this PR
🤖 Generated with [Claude Code](https://claude.com/claude-code)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: resolve 22 Qodana warnings from CI analysis [struts-intellij-plugin]
Posted by "lukaszlenart (via GitHub)" <gi...@apache.org>.
lukaszlenart merged PR #52:
URL: https://github.com/apache/struts-intellij-plugin/pull/52
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: resolve 22 Qodana warnings from CI analysis [struts-intellij-plugin]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #52:
URL: https://github.com/apache/struts-intellij-plugin/pull/52#issuecomment-3953449005
# Qodana 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
<details>
<summary>View the detailed Qodana report</summary>
To be able to view the detailed Qodana report, you can either:
- Register at [Qodana Cloud](https://qodana.cloud/) and [configure the action](https://github.com/jetbrains/qodana-action#qodana-cloud)
- Use [GitHub Code Scanning with Qodana](https://github.com/jetbrains/qodana-action#github-code-scanning)
- Host [Qodana report at GitHub Pages](https://github.com/JetBrains/qodana-action/blob/3a8e25f5caad8d8b01c1435f1ef7b19fe8b039a0/README.md#github-pages)
- Inspect and use `qodana.sarif.json` (see [the Qodana SARIF format](https://www.jetbrains.com/help/qodana/qodana-sarif-output.html#Report+structure) 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:
```yaml
- name: 'Qodana Scan'
uses: JetBrains/qodana-action@v2025.3.1
with:
upload-result: true
```
</details>
<details>
<summary>Contact Qodana team</summary>
Contact us at [qodana-support@jetbrains.com](mailto:qodana-support@jetbrains.com)
- Or via our issue tracker: https://jb.gg/qodana-issue
- Or share your feedback: https://jb.gg/qodana-discussions
</details>
<!-- JetBrains/qodana-action@v2025.3.1 : Qodana for JVM, -->
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
Re: [PR] fix: resolve 22 Qodana warnings from CI analysis [struts-intellij-plugin]
Posted by "github-actions[bot] (via GitHub)" <gi...@apache.org>.
github-actions[bot] commented on PR #52:
URL: https://github.com/apache/struts-intellij-plugin/pull/52#issuecomment-3953408143
<!-- plugin-artifact-comment -->
🔌 **Plugin artifact ready for testing!**
Download from [Actions artifacts](https://github.com/apache/struts-intellij-plugin/actions/runs/22360411779#artifacts)
Artifact: `struts2-253.18970.1`
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: notifications-unsubscribe@struts.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org