SoftwareUnderstanding / inspect4py

Static code analysis package for Python repositories
https://inspect4py.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
27 stars 10 forks source link

A few new features and improvements to inspect4py #356

Closed lazyhope closed 2 years ago

lazyhope commented 2 years ago

In this PR a few new features are added, including:

There are also some improvements to inspect4py, including:

dgarijo commented 2 years ago

@zl81, thanks, this looks like great progress for inspect4py. I need a few days for reviewing the PR. I will produce a review then

lazyhope commented 2 years ago

Thank you, please take your time!

rosafilgueira commented 2 years ago

Many thanks @zl81 !

dgarijo commented 2 years ago

Next time @zl81 please push your changes to the dev branch. Thanks!

lazyhope commented 2 years ago

Next time @zl81 please push your changes to the dev branch. Thanks!

No problem!

lazyhope commented 2 years ago

looks ok to me. I have to pass the tests locally (I have not set up the actions yet)

@dgarijo I have left one test unpassed intentionally, this unpassed test shows an open issue of our current implementation treating class instantialization calls the same as regular function calls, right now we don't have a good way to distinguish between the two.

dgarijo commented 2 years ago

Ok. Can you please open an issue? It's easier to track open problems this way.

lazyhope commented 2 years ago

Ok. Can you please open an issue? It's easier to track open problems this way.

Sure, I will open it tomorrow as it's late in my current time zone.

dgarijo commented 2 years ago

Thanks! No rush

El vie., 19 ago. 2022 5:43 p. m., zl81 @.***> escribió:

Ok. Can you please open an issue? It's easier to track open problems this way.

Sure, I will open it tomorrow as it's late in my current time zone.

— Reply to this email directly, view it on GitHub https://github.com/SoftwareUnderstanding/inspect4py/pull/356#issuecomment-1220821509, or unsubscribe https://github.com/notifications/unsubscribe-auth/AALTIGQ2G3W2KASIHPTCVA3VZ6TRVANCNFSM56SGRRAQ . You are receiving this because you were mentioned.Message ID: @.***>