lcpp-org / crane

A MOOSE application dedicated to general Chemical ReAction NEtworks for plasma chemistry and thermochemistry problems.
https://crane-plasma-chemistry.readthedocs.io/
GNU Lesser General Public License v2.1
21 stars 20 forks source link

Make sure overload resolution occurs with the right getValue #110

Closed cticenhour closed 1 year ago

cticenhour commented 1 year ago

One more fixup here @smpeyres, as I missed this in the previous PR (only shows up in debug mode). Ready for review and merge.

refs shannon-lab/zapdos#203

smpeyres commented 1 year ago

@cticenhour Commit needs to be signed before merge

cticenhour commented 1 year ago

Will look into it later today. Last PR I just pushed and it was signed, but this one did not perform the same...

cticenhour commented 1 year ago

@smpeyres Alright, we're ready to merge. Pushed originally from a Linux server that didn't have my signature attached to it.