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.

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

validateProcesses.py adding a quick fix while searching ProcessEvent method #270

Merged
merged 1 commit into from
Aug 25, 2022

Conversation

jgalan
Copy link
Member

@jgalan jgalan commented Jul 26, 2022

jgalan Ok: 1

The validateProcesses.py is failing on the pipeline validation: rest-for-physics/axionlib#12

The problem is that the keyword ProcessEvent appears in the documentation of TRestAxionEventProcess but in reality there is no ::ProcessEvent implementation. I added a more strict method search inside the script.

@jgalan jgalan requested review from nkx111, lobis and a team July 26, 2022 11:39
@jgalan jgalan requested a review from a team July 27, 2022 07:26
@jgalan
Copy link
Member Author

jgalan commented Jul 29, 2022

Approuve plz!

@jgalan jgalan merged commit ff5392f into master Aug 25, 2022
@jgalan jgalan deleted the validate_processes_fix branch August 25, 2022 11:38
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.

None yet

2 participants