Integration of LArSoft v10 geometry, vetted [2/2]#808
Conversation
|
trigger build SBNSoftware/icarusalg#92 LArSoft/lar*@LARSOFT_SUITE_v10_04_04 |
|
✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard |
|
❌ CI build for ICARUS Failed at phase build ICARUS on slf7 for c14:prof -- details available through the CI dashboard 🚨 For more details about the failed phase, check the build ICARUS phase logs parent CI build details are available through the CI dashboard |
|
✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard |
|
❌ CI build for ICARUS Failed at phase ci_tests ICARUS on slf7 for e26:prof - ignored warnings for build -- details available through the CI dashboard 🚨 For more details about the failed phase, check the ci_tests ICARUS phase logs parent CI build details are available through the CI dashboard |
|
trigger build SBNSoftware/icarusalg#92 LArSoft/lar*@LARSOFT_SUITE_v10_04_04 |
|
✔️ CI build for LArSoft Succeeded on slf7 for c14:prof -- details available through the CI dashboard |
|
❌ CI build for ICARUS Failed at phase build ICARUS on slf7 for c14:prof -- details available through the CI dashboard 🚨 For more details about the failed phase, check the build ICARUS phase logs parent CI build details are available through the CI dashboard |
|
✔️ CI build for LArSoft Succeeded on slf7 for e26:prof -- details available through the CI dashboard |
|
🚨 For more details about the warning phase, check the ci_tests ICARUS phase logs parent CI build details are available through the CI dashboard |
jzennamo
left a comment
jzennamo
left a comment
There was a problem hiding this comment.
I enthusiastically approve!
5 participants
This branch is the merge of the proposed
geom-separateby @knoepfel and ICARUS changes, and it supersedes PR #642.I "self-approve" ✔️ the PR as it is the result of the review of PR #642. However, a formal approval is required by GitHub.
This PR depends on SBNSoftware/icarusalg#92.