tldahlgren reviewed a spack/spack pull request
(Re-)Confirmed branch and all 57 version sha256….
rashawnLK reviewed a spack/spack pull request
Thanks Wil! This looks good….
zackgalbreath reviewed a spack/spack pull request
Thanks for fixing this so quickly!…
adamjstewart reviewed a spack/spack pull request
Solves the issue I reported on Slack. Not confident enough in this part of Spack to pull the trigger and merge. Would be great to get reviews from others….
mlstowell reviewed a mfem/mfem pull request
There are still a couple outstanding small comments but I’m sure they will be addressed. Thanks @dylan-copeland! This is a useful tool (and a challenge to other developers to add mixed element support). …
waynemitchell reviewed a hypre-space/hypre pull request
Looks good except my one minor comment above. Thanks, @oseikuffuor1 !…
rfalgout reviewed a hypre-space/hypre pull request
Looks good modulo our discussion about size_in
. Thanks!
…
grondo reviewed a flux-framework/flux-core pull request
This LGTM. Nice job catching the exception note corner case!…
pelesh reviewed a LLNL/hiop pull request
This branch is well tested and ready to merge. It might be a good idea to get the updated version of Ginkgo on all CI pipelines first….
agcapps reviewed a LLNL/RAJA pull request
Looks good…
agcapps reviewed a LLNL/conduit pull request
Thanks, Cyrus….
agcapps reviewed a LLNL/CARE pull request
If this fixes CARE’s GitHub CI, that’s what we need. But I’m still curious about the possibility of the fix (jwlawson?) going away. (Maybe that concern isn’t warranted.)…
rfalgout reviewed a XBraid/xbraid pull request
Looks good, Jacob! …