Skip to content
New issue

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

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

Already on GitHub? # to your account

Rename unittests to unittests_edm4hep #226

Merged
merged 1 commit into from
Sep 13, 2023
Merged

Rename unittests to unittests_edm4hep #226

merged 1 commit into from
Sep 13, 2023

Conversation

jmcarcell
Copy link
Member

@jmcarcell jmcarcell commented Sep 13, 2023

BEGINRELEASENOTES

  • Rename the cmake executable or target unittests to unittests_edm4hep, to avoid possible collisions since the unittests name is relatively common

ENDRELEASENOTES
I don't mind the name, it just has to be a different one from unittests. Similar to AIDASoft/podio#481

@tmadlener tmadlener enabled auto-merge (squash) September 13, 2023 07:00
@jmcarcell jmcarcell disabled auto-merge September 13, 2023 07:05
@jmcarcell jmcarcell changed the title Rename unittests to unittests_edm4hep Rename unittests to unittests_edm4hep Sep 13, 2023
@jmcarcell jmcarcell merged commit cae7a4c into main Sep 13, 2023
@jmcarcell jmcarcell deleted the unittests branch September 13, 2023 07:05
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants