equinor / xtgeo

XTGeo Python class library for subsurface Surfaces, Cubes, Wells, Grids, Points, etc
https://xtgeo.readthedocs.io/en/latest/
GNU Lesser General Public License v3.0
109 stars 56 forks source link

CLN: Remove bare xtgeo imports #1161

Open mferrera opened 5 months ago

mferrera commented 5 months ago

Resolves #1149

I was a bit opinionated with these changes in expanding import paths fully and using relative imports, but I don't particularly believe that needs to be enforced and no linting rules exist to enforce it anyway. I do think that it will be a bit helpful in the near term, with structural refactors ahead, and that we should discuss whether or not to codify it in the style guide. Some major projects, e.g. pandas, do this.

This also adds a manual check in linting ci to make sure no further import xtgeos are added