Skip to content

[PWGJE] Implemented suggestions from comments on previous pull request.#16002

Closed
Lolle2001 wants to merge 2 commits intoAliceO2Group:masterfrom
Lolle2001:dev
Closed

[PWGJE] Implemented suggestions from comments on previous pull request.#16002
Lolle2001 wants to merge 2 commits intoAliceO2Group:masterfrom
Lolle2001:dev

Conversation

@Lolle2001
Copy link
Copy Markdown

  • Updated the includes according to the O2 guidelines.
  • Removed registry as an argument in the analyse methods.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 28, 2026

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

@github-actions github-actions Bot changed the title Implemented suggestions from comments on previous pull request. [PWGJE] Implemented suggestions from comments on previous pull request. Apr 28, 2026
@Lolle2001 Lolle2001 closed this Apr 29, 2026
@Lolle2001 Lolle2001 reopened this Apr 29, 2026
- Updated the includes according to the O2 guidelines.
- Removed registry as an argument in the `analyse` methods.
@vkucera
Copy link
Copy Markdown
Collaborator

vkucera commented Apr 29, 2026

@Lolle2001 Please avoid fragmenting PRs. Update the existing one instead. #15914

@vkucera vkucera closed this Apr 29, 2026
@Lolle2001
Copy link
Copy Markdown
Author

Lolle2001 commented Apr 29, 2026

Hi @vkucera, @pdhankhe will close the one from before.

EDIT: @pdhankhe has closed it, could you please open this one, since I will be the main contributor to this code.

@Lolle2001
Copy link
Copy Markdown
Author

@Lolle2001 Please avoid fragmenting PRs. Update the existing one instead. #15914

Hi @vkucera, could you reopen this PR please?

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