The Wayback Machine - http://web.archive.org/web/20200919115709/https://github.com/pahen/madge/pull/251
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

Generate graph for only circular dependencies #251

Conversation

@RamarajaRamanujan
Copy link

RamarajaRamanujan commented Jun 30, 2020

This PR is inspired by #224
It wasn't approved and merged as it had linting issues and a few conflicts.

I have fixed both, refactored the code and submitted the changes to the original PR as seen here,
gaspardip#2
but it's been almost 3 months and the author hasn't responded.

So, I have decided to submit the required changes directly as a separate PR.
gaspardip still gets credit as I have preserved their commit in this PR. You can see that they are included as a participant.

@RamarajaRamanujan RamarajaRamanujan changed the title Feature/add circular graph Generate graph for only circular dependencies Jun 30, 2020
@Hari-Soni
Copy link

Hari-Soni commented Aug 31, 2020

HI,

Will it be possible to merge the PR and push a new version.

@pahen
Copy link
Owner

pahen commented Sep 14, 2020

Fixed in #224

@pahen pahen closed this Sep 14, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants
You can’t perform that action at this time.