svalinn / DAGMC

Direct Accelerated Geometry Monte Carlo Toolkit
https://svalinn.github.io/DAGMC
Other
96 stars 61 forks source link

Adding reorder flag to CI builds #902

Closed pshriwise closed 11 months ago

pshriwise commented 11 months ago

Description

@aprilnovak discovered an issue with the initialization order of the DagMC_Logger class and I believe has found some similar issues previously in DAGMC/MOAB. This adds the -Wreorder flag to our CI builds in the CMake command to ensure we catch this in the future.

image

Motivation and Context

Ensure that future changes/additions to the code are easy to incorporate into MOOSE applications.