Release 1.12.5 Requires a TOML Library about pydeps HOT 6 CLOSED

I've fixed the crashing on missing config in config files, and the two most obvious import-error issues.
Version 1.12.8 is available on PyPI.

from pydeps.

No need to apologize! I was just trying to point out that this is a much bigger issue than the title implies (but you correctly point out in your report).

from pydeps.

I think the title here is burying the lede a bit:

Even when tomlkit is installed manually, if you haven't correctly set up a pydeps config file it will also fail:

I came here looking for other issues reported about this since that's what I was seeing in version 1.12.7 so it hasn't been fixed yet.

from pydeps.

@mivade apologies. The lack of tomlkit was the first issue I hit, and I started the issue. Trying to fix it, I hit more and more until I determined that the release was basically completely broken based on the documentation at the time.

The title could very well have been "Release 1.12.5 doesn't work at all by following docs"

from pydeps.

I see there is a problem if a config file is found and doesn't contain a pydeps config (configs.pyL365).

Hmm.. I tried to make it so (a) pythons with included tomllib would work, (b) any of the other common(?) toml-libraries (i.e. tomlkit and toml) would work if they were found.

My thinking being that people who wanted to pick up configs from pyproject.toml already had their preferred toml-library installed.

Is there a better way of doing this?

from pydeps.

@thebjorn thank you! A PR was opened in our codebase this morning via Renovate and it's working just as before. 🎉

from pydeps.