No specific issue being addressed here, just some quality of life changes to some of our spack scripts. This makes it so that the spack environment is created in a single line, and that there is no longer a module purge that is ran before config is loaded. This is helpful to prevent harmful environment modifications by our scripts where they might be unnecessary.
If these changes are ok, we can extend this to other clusters as needed. Many of the whitespace changes were automatically made by my editor, and we can change / revert as needed. We don't really have any bash linting configuration or developer guidelines surrounding those files...
Merge request type
Relates to
This MR updates
Summary
No specific issue being addressed here, just some quality of life changes to some of our spack scripts. This makes it so that the spack environment is created in a single line, and that there is no longer a
module purge
that is ran before config is loaded. This is helpful to prevent harmful environment modifications by our scripts where they might be unnecessary.If these changes are ok, we can extend this to other clusters as needed. Many of the whitespace changes were automatically made by my editor, and we can change / revert as needed. We don't really have any bash linting configuration or developer guidelines surrounding those files...