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

dependency list and reverse dependency list mismatch #764

Closed
szabgab opened this issue Feb 5, 2013 · 4 comments
Closed

dependency list and reverse dependency list mismatch #764

szabgab opened this issue Feb 5, 2013 · 4 comments

Comments

@szabgab
Copy link
Contributor

szabgab commented Feb 5, 2013

Test-DependentModules lists Moose in its reverse dependency list, but in Moose it is only listed in one of the xt/ files and it is not listed as one of the dependencies of Moose.
The latter is the correct,

Maybe the reverse dependency should have two sections

  1. for distributions that really depend on this one
  2. distributions that optionally (maybe in a test) use it
@rwstauner
Copy link
Contributor

Thanks for catching this!
The api query doesn't currently restrict by "phase".
It probably should, but it might be nice if we also provided an option for showing all (or filtering by phase)
because most "Test::" modules would never be found.

@rwstauner
Copy link
Contributor

I started doing some work toward this and realized I need to increase my ES-fu with some sort of nested or other query. I'm planning on doing a lot of reading this week so maybe I'll figure something out.

@monken
Copy link
Contributor

monken commented Mar 25, 2013

Moose doesn't list it because it's in the testing_requires category. We don't show these because they are not as interesting as the rest.
The reverse dependencies show them however, mostly to get an accurate count. Adding an option to filter by phase is certainly a good idea but I don't think the current behavior is broken.

@ranguard
Copy link
Member

Moved to wishlist https://github.com/CPAN-API/cpan-api/wiki/Wishlist

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

No branches or pull requests

4 participants