Closed adayton1 closed 8 months ago
Hi @adayton1 . In your last MR, https://github.com/LLNL/CARE/pull/245, adriene bernede suggested a few changes, but it was too late to include them since you already merged.
Maybe fix the license mistake that adriene found there (files that mention the CHAI license instead of the care one due to a copy/paste issue)?
Hi @adayton1 . In your last MR, #245, adriene bernede suggested a few changes, but it was too late to include them since you already merged.
Maybe fix the license mistake that adriene found there (files that mention the CHAI license instead of the care one due to a copy/paste issue)?
I'm going to take a pass at updating all the copyright dates in a follow on merge request, and I'll fix the license mistake in that branch.
@adayton1 I think we should consider placing your custom CARE package in the RADIUSS Spack Configs repo. That’s where we centralize the local spack packages. Would you be fine with that ? Not a high priority item, but I’d like your agreement for when the time comes.