Crash when running snakefmt 0.1.4

[kpj]

I am using snakefmt v0.1.4 on macOS 10.15.6. When simply running the script, it crashes:

$ snakefmt
Traceback (most recent call last):
  File "/Python/3.8/bin/snakefmt", line 5, in <module>
    from snakefmt.snakefmt import main
  File "/Python/3.8/lib/python/site-packages/snakefmt/__init__.py", line 6, in <module>
    __version__ = toml.load(project_fpath)["tool"]["poetry"]["version"]
  File "/Python/3.8/lib/python/site-packages/toml/decoder.py", line 133, in load
    with io.open(_getpath(f), encoding='utf-8') as ffile:
FileNotFoundError: [Errno 2] No such file or directory: '/Python/3.8/lib/python/site-packages/pyproject.toml'

This seems to be due to the following two lines:

snakefmt/snakefmt/__init__.py

Lines 5 to 6 in 25c046b

	project_fpath = Path(__file__).parent.parent / "pyproject.toml"
	__version__ = toml.load(project_fpath)["tool"]["poetry"]["version"]

[mbhall88]

Yes, I'm also having this problem in bioconda/bioconda-recipes#24238. @bricoletc I think we might need to change the way the version number is gathered back to what it was before?

[bricoletc]

Thanks for this @kpj , I should totally have looked into this when trying to have single version specification in pyproject.toml.
Problem is pip install does not distribute pyproject.toml. This was really handy to find a fix: python-poetry/poetry#273
Running poetry build to distribute package creates PKG_INFO, pyproject.toml and setup.py files which all have the version from pyproject.toml. Then if you distribute sources and pip install you get a .dist-info directory which contains a METADATA file and that has version in it.

I added fix recommended in python-poetry/poetry#273 by using importlib.metadata.version

I confirmed this works from both pip install snakefmt and poetry install from a local copy of the sources.

@mbhall88 i almost feel like we need to push this change to test pypi or something and pip install to confirm this works?

[bricoletc]

@mbhall88 importlib.metadata was only added to Python standard library in 3.8 so my proposed fix now uses a backport called importlibs_metadata. This adds a dependency to snakefmt but I quite like it, you can do a lot more package introspection than just getting the version

[bricoletc]

This seems to have worked, closing!