INL's continuous integration (CI) system CIVET allows for testing of external MOOSE applications like CRANE, both within their own repository (PRs, branch merges, etc.) and as part of regular MOOSE PR testing - see this page for a recent set of results on the MOOSE next branch. CRANE could benefit from this workflow, given as up until now CRANE components have been tested only locally and as part of their usage in Zapdos.
Another change I would make would be the creation of a devel branch. After a PR merge, everything would be re-tested and then merged into master only when everything is passing. The branch testing would be where you might have extended testing in the future.....at first it might have the same test configurations - we call them "recipes" - as your PRs. This workflow protects your master branch from possible failure points that might occur when multiple people are working on the code. Having a devel branch is optional, but all of the INL MOOSE apps and several external apps use this structure.
To get this rolling, we would need to perform a few things:
Happy to answer any questions you might have on this!
Tagging @smpeyres @dcurreli
INL's continuous integration (CI) system CIVET allows for testing of external MOOSE applications like CRANE, both within their own repository (PRs, branch merges, etc.) and as part of regular MOOSE PR testing - see this page for a recent set of results on the MOOSE
nextbranch. CRANE could benefit from this workflow, given as up until now CRANE components have been tested only locally and as part of their usage in Zapdos.Another change I would make would be the creation of a
develbranch. After a PR merge, everything would be re-tested and then merged intomasteronly when everything is passing. The branch testing would be where you might have extended testing in the future.....at first it might have the same test configurations - we call them "recipes" - as your PRs. This workflow protects yourmasterbranch from possible failure points that might occur when multiple people are working on the code. Having adevelbranch is optional, but all of the INL MOOSE apps and several external apps use this structure.To get this rolling, we would need to perform a few things:
Happy to answer any questions you might have on this!
Tagging @smpeyres @dcurreli