Fix finding packages without an __init__.py#1333
Fix finding packages without an __init__.py#1333Pierre-Sassoulas merged 2 commits intopylint-dev:mainfrom
__init__.py#1333Conversation
|
If you have a pointer to how to approach testing asteroid/pylint, in general, I might be able to help. It wouldn’t be too challenging to generate a temp directory that reproduces the problem. Otherwise, I can write a test method the way I normally would that you can adapt? |
|
For some reason another test broke as well. I have got some spare time now, so I'll take a look myself. Thanks anyway 😄 |
|
While I got the test suite to fail, I can't get the added test to fail without the change (thus it doesn't test anything..). Would need to do some further investigation into what happened before the fauly commit in this function and what is happening now. |
I think recreating a temp directory that reproduce the issue is the way to go here. Thank you for your help, much appreciated !
Could the problem come from the fact we removed the : |
|
Test now fails with previous code and passes with new code. The mistake was that we removed |
Pierre-Sassoulas
left a comment
There was a problem hiding this comment.
Let's release 2.9.3 with this.
Steps
Description
Closes #1327
Had no time to write tests yet, so if somebody can come up with something good: please take over this PR, you're more than welcome to.
Just wanted to illustrate that this is what should be added again.
This regression was introduced in 2ee20cc
Type of Changes
Related Issue
@Pierre-Sassoulas, this might warrant a
2.9.3release.