Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

ctp config to ccdb #8595

Merged
merged 4 commits into from
Apr 21, 2022
Merged

ctp config to ccdb #8595

merged 4 commits into from
Apr 21, 2022

Conversation

lietava
Copy link
Contributor

@lietava lietava commented Apr 15, 2022

No description provided.

@lietava lietava marked this pull request as ready for review April 18, 2022 13:24
@lietava lietava requested review from peressounko, kharlov and a team as code owners April 18, 2022 13:24
@lietava
Copy link
Contributor Author

lietava commented Apr 19, 2022

Hi @peressounko , @kharlov , I was looking in the log and can not see anything related to CTP. Please, let me know if there is any issue with my code.

@peressounko
Copy link
Collaborator

dear @lietava, this is clang formatting issue, nothing to do with code itself. OK from PHOS side. Dmitri

@lietava
Copy link
Contributor Author

lietava commented Apr 20, 2022

HI @shahor02, what is holding this request ?

@shahor02 shahor02 merged commit 143c7ff into AliceO2Group:dev Apr 21, 2022
@lietava
Copy link
Contributor Author

lietava commented Apr 21, 2022

Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@kharlov kharlov kharlov approved these changes

@shahor02 shahor02 shahor02 approved these changes

@peressounko peressounko Awaiting requested review from peressounko peressounko is a code owner

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@lietava @peressounko @shahor02 @kharlov