a-scie / lift

Provides science, a high level tool to build scies with.
https://science.scie.app/
Apache License 2.0
12 stars 5 forks source link

Undo tomllib `no-infer-dep` #1

Closed jsirois closed 1 year ago

jsirois commented 1 year ago

Right here: https://github.com/a-scie/lift/blob/24f399503875ce937224afc6e0dfa550b5cb8fa4/science/config.py#L4-L6

The switch from tomli to the stdlib occurred in https://github.com/a-scie/lift/commit/24f399503875ce937224afc6e0dfa550b5cb8fa4. That also upgraded to Pants 2.15.0rc2. When we can upgrade to a version of Pants that fixes this lack of knowledge of the tomllib stdlib, we can undo the no-infer-dep.

jsirois commented 1 year ago

The Pants issue tracking this is here: https://github.com/pantsbuild/pants/issues/18055

jsirois commented 1 year ago

Killed this comment in 0489944e929ef511beabaa9334364ef2e495b8d0 since it's no longer required with the switch from Pants to Nox.