Daniël van Noord

Results 669 comments of Daniël van Noord

I have a fix for this! I'm just going through some old issues to see if there is a reason for the current behaviour.

This issue is quite old: https://github.com/PyCQA/astroid/issues/135 One of the problems is that `pylint` is heavily dependent on comprehension nodes being `Uninferable`. Just adding a `infer()` method to `ComprehensionScope` made 10+...

I have played around with `astroid` and `pylint` and came up with a potential fix. You can see it in #1475.

> I guess next step is to see what fails / depends on the current behavior. Commenting out the relevant lines crashes already in the bootstrapping of the `str` `ClassDef`....

> I think it might only be a "gotcha" when creating fake nodes in the brain. When creating a fake ClassDef from scratch in the namespace brain, there's no logical...

I have investigated this a little. ```python def test_conditional_definition_partial_pickle(self) -> None: """Regression test for a conditional defintion of a partial function wrapping pickle. Reported in https://github.com/PyCQA/astroid/issues/925 """ module = resources.build_file(...

I'd like to get #1598 merged before we do any additional work on `Enums`.

@DetachHead Please use the issue template when filing issues, even if you have opened the issue before. Issues like this are not likely to be taken up by anyone..

> * Or just bump the min requirement to `1.14`. `1.14` is compatible with Python `>=3.5` so that wouldn't be an issue. See https://github.com/PyCQA/astroid/pull/1745

@cdce8p Gentle ping as this is blocking some other typing work