Michele Lacchia

Results 102 comments of Michele Lacchia

@ghpqans Thanks for the report. It seems more recent builds are failing due to excessive memory consumption, i.e. the issue outlined here: https://github.com/readthedocs/readthedocs.org/issues/6412 I need to look into why this...

@BlaneG Hi Blane, thanks for the contribution! I was not aware of such addition in Python 3.8, I will check it out. The idea is definitely interesting. It's something I...

I think the idea is good. Now besides the minor comments I added above, two things are needed mainly: 1) enable all the tests and ensure everything is passing 2)...

Hi @BlaneG, in order to have this PR merged, you need to fix the conflicts and ensure all tests are passing. Are you still working on it?

Thanks for the bug report @Mahdi-py. Indeed, this is due to the way the AST visitor is implemented. I'll take a look at a possible fix in the near future.

That fix would allow Radon to count the complexity of all the code within the nested classes. That's the correct thing to do. However, it won't cause the nested classes...

@rwbogl Yes, that would be a good idea. It's not super simple because the error is caught in `radon/__init__.py` and there is no CLI information. A hack would be to...

Sorry for the delay. This is not currently supported, so it would require a change in the parsing code. At the moment I cannot do it, but I will review...

@psychemedia That's interesting. I would say they are lines of code. Maybe not Python code, but code nonetheless. Determining if they contain real Python code might be tricky though. How...

@dollodart Hi David, I missed this message and I see it only now. I'd say such a visitor would fit within the scope of the project, as it provides metrics...