Instead of using NULL as the default and then checking if it is NULL and then causing something else to happen the user should be aware of what the real default is, which is "metric ton" instead of the end result being hidden from them. If the default really is NULL then nothing should be done. Referring to the following hidden change
https://github.com/nmfs-ost/satf/blob/1641eba94dc762663725213503d450f984bec846/R/plot_spawning_biomass.R#L38-L42
Instead of using NULL as the default and then checking if it is NULL and then causing something else to happen the user should be aware of what the real default is, which is
"metric ton"instead of the end result being hidden from them. If the default really is NULL then nothing should be done. Referring to the following hidden changehttps://github.com/nmfs-ost/satf/blob/1641eba94dc762663725213503d450f984bec846/R/plot_spawning_biomass.R#L38-L42