Skip to content

[PWGJE] fix code-check issues in QG ML producer - #17505

Merged
nzardosh merged 2 commits into
AliceO2Group:masterfrom
mulewicz:fix-qgml-code-check
Aug 19, 2026
Merged

[PWGJE] fix code-check issues in QG ML producer#17505
nzardosh merged 2 commits into
AliceO2Group:masterfrom
mulewicz:fix-qgml-code-check

Conversation

@mulewicz

Copy link
Copy Markdown
Contributor

Follow-up to #17400 fixing the code-check issues reported after the merge.

Changes:

  • add the direct Framework/AnalysisHelpers.h include required for Produces
  • mark hasITSLayerHit as [[nodiscard]]

The O2 linter passes locally.

@github-actions

Copy link
Copy Markdown

O2 linter results: ❌ 0 errors, ⚠️ 0 warnings, 🔕 0 disabled

@github-actions github-actions Bot changed the title PWGJE: fix code-check issues in QG ML producer [PWGJE] fix code-check issues in QG ML producer Aug 19, 2026
@mulewicz
mulewicz marked this pull request as draft August 19, 2026 09:30
@nzardosh

Copy link
Copy Markdown
Collaborator

please let me know if this is intended to be in draft or if it should be reviewed

@mulewicz
mulewicz marked this pull request as ready for review August 19, 2026 11:16
@mulewicz

Copy link
Copy Markdown
Contributor Author

please let me know if this is intended to be in draft or if it should be reviewed

It's ready for the review, thank you

@nzardosh
nzardosh merged commit 3f30a35 into AliceO2Group:master Aug 19, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Development

Successfully merging this pull request may close these issues.

2 participants