Currently, module weight is explicitly set in Insert and its sub-modules. Setting module weight is a legacy from the module's D7 version and actually does not seem to be required on the module's D8 version as there do not seem to be hooks that need to be processed in a particular order. Apart from that, setting module weight is done on installation only, which is not a solid solution.
Having the actual dependencies required per insert.info, should just be sufficient for Insert, while sub-modules can make use of hook_module_implements_alter() instead of setting module weight on installation.

Comments

Snater created an issue. See original summary.

  • Snater committed ffef58e on 8.x-2.x 
    Issue #3022983: Removed setting module weight
snater’s picture

snater commented
Status: Active » Fixed

All Insert tests pass, so it should be fine to have all that module weight setting on installation removed.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.