Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

not detecting various imports and missing cycles #5

Open
irmen opened this issue Feb 12, 2017 · 1 comment
Open

not detecting various imports and missing cycles #5

irmen opened this issue Feb 12, 2017 · 1 comment
Labels

Comments

@irmen
Copy link

irmen commented Feb 12, 2017

Hi,
it seems the tool doesn't process all the import syntax correctly. Not sure which ones it skips but I recommend checking into it:

import Package [as newname]
from Package import sub [as newname]
from .. import relativething [as newname]
from Foo import *

I don't know if the cycle detection is buggy or that the tool misses some imports (or both), but it doesn't find any cycle at all in my Pyro4 project. But there sure are cycles! See irmen/Pyro4#148 (the graphs are generated by my own cycle analisys tool + graphviz btw)

@bndr bndr added the bug label Feb 12, 2017
@bndr
Copy link
Owner

bndr commented Feb 12, 2017

Thanks for reporting! I'll take a look

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants