search

snerstack / sner-monorepo

sner -- slow network recon
1 stars 0 forks source link

server: pylint cyclic-import detection buggy (temporarily disabled) #56

Open bodik opened 5 months ago

bodik commented 5 months ago

resolve if possible

Commit: 53c6860b10358594ff2bf26f3416594a3a424dff
server: disable buggy cyclic imports detection for py3.11 + pylint 3.1.0
bodik commented 5 months ago
bodik commented 4 months ago

bodik — 05/03/2024 11:13 AM hi, would anyone give a tip how to hunt for mystery cyclic-imports reports which are reported only on some pylint runs but not for others while the code does not change ? (pylint 3.1.0, py 3.11) -- ci failed [https://github.com/snerstack/sner-monorepo/actions/runs/8936689485/attempts/1] ci passed [https://github.com/snerstack/sner-monorepo/actions/runs/8936689485/attempts/2] jacobtylerwalls — 05/04/2024 4:21 PM Nothing comes to mind, sorry. We do have another report of indeterminate cyclic-import output here. https://github.com/pylint-dev/pylint/issues/9168 bodik — 05/04/2024 4:26 PM jacobtylerwalls thanks, i've found that, it does not look like my code, but i'll try to dig in deeper jacobtylerwalls — 05/04/2024 4:29 PM Because it's import related, it's going to interact pretty tightly with astroid. Assuming you can repro the indeterminancy (big if...) maybe you can hack on places like _add_imported_module() or hack out some of the module caching in astroid We added AstroidManager.clear_cache() to show all the places you might need to consider (shudder) but likely the only that matters for you is from astroid.interpreter._import.spec import _find_spec