Following discussions with @seddonym. A suggested renaming:
grimp.Level to grimp.Layer - since a "level" is actually a layer (these terms are currently mixed and used to mean the same thing, lets just use one terminology).
grimp.Level.layers to grimp.Layer.module_tails - since there are no "layers" within a layer/level, the items within a layer are actually modules (or the "tails" of modules, in the case of containers).
Right now this is just the python changes - we should probably align the rust code too, although for this PR these python changes might be enough, so that we can prioritise tidying the package public API (so it is publishable) and also unblock https://github.com/seddonym/import-linter/pull/209.
seddonym
commented
5 months ago
Following discussions with @seddonym. A suggested renaming:
grimp.Leveltogrimp.Layer- since a "level" is actually a layer (these terms are currently mixed and used to mean the same thing, lets just use one terminology).grimp.Level.layerstogrimp.Layer.module_tails- since there are no "layers" within a layer/level, the items within a layer are actually modules (or the "tails" of modules, in the case ofcontainers).Right now this is just the python changes - we should probably align the rust code too, although for this PR these python changes might be enough, so that we can prioritise tidying the package public API (so it is publishable) and also unblock https://github.com/seddonym/import-linter/pull/209.