databrickslabs / ucx

Automated migrations to Unity Catalog
Other
239 stars 85 forks source link

[BUG]: Direct File System Access paths sometimes contain the word 'Uninferable' #3040

Open ericvergnaud opened 1 month ago

ericvergnaud commented 1 month ago

Is there an existing issue for this?

Current Behavior

When linting code, Advices may contain messages that contain the word 'Uninferable', which is cryptic since it does not refer to an existing path.

Expected Behavior

The messages should not contain the word 'Uninferable'

Steps To Reproduce

No response

Cloud

AWS

Operating System

macOS

Version

latest via Databricks CLI

Relevant log output

No response

ericvergnaud commented 1 month ago

See https://github.com/pylint-dev/astroid/pull/2622

ericvergnaud commented 1 week ago

A bug fix is about to be integrated, see https://github.com/pylint-dev/astroid/pull/2622. But this might break our code a bit because we will no longer be seeing partially inferred f-strings. A workaround would be to provide our own replacement for JoinedStr, see https://github.com/pylint-dev/astroid/pull/2622#issuecomment-2481277493.