No functional difference in this PR, only code transformation, module reorg, and inlining. Ultimate goals:
remove named class instances (this is straightforward)
get rid of utility functions that we don't use anywhere hence actually have no utility
get rid of unnecessary split between Internal and not. There are really no Internal modules here anyway, everything makes explicit assumptions about the project structure, file structure, hierarchy, etc. It makes more sense to just expose the Test module and some kind of DeployScript rexports to facilitate writing deployments and isolating some code as "this is all you will need as a library writer"
inline almost all of Utils.Errors : These combinators do not pass whatever threshold that is, with perhaps the exception of withExceptT'. It's more of an indication that the code is structured poorly that there don't exist any anlogues of these functions in other purescript libs, even Control.Error.Utils.
No functional difference in this PR, only code transformation, module reorg, and inlining. Ultimate goals:
Internaland not. There are really noInternalmodules here anyway, everything makes explicit assumptions about the project structure, file structure, hierarchy, etc. It makes more sense to just expose theTestmodule and some kind ofDeployScriptrexports to facilitate writing deployments and isolating some code as "this is all you will need as a library writer"withExceptT'. It's more of an indication that the code is structured poorly that there don't exist any anlogues of these functions in other purescript libs, evenControl.Error.Utils.