-
Notifications
You must be signed in to change notification settings - Fork 8
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
Account for modules in the no_index section of META.yml #4
Conversation
…rked as no-index by the author.
Thank you for your feedback! I'll check it. |
Hi. If you want it, I will merge this request soon.
Again, if you want it, I will merge this request soon. |
Hi Yasutaka, I think your two points are great ideas, I'm going to make the changes and edit my pull request to include them. I will try to have both ready for you this weekend. Thank you! |
How about the status? |
Hi Yasutaka, My apologies, I haven't had as much free time as I expected in the last couple of weeks. I have implemented |
Hi Guillaume. No problem.
Great. I'm looking forward to your update. Thanks. |
Hi Yasutaka, Thank you for your patience. I just pushed two commits that generalize the code to handle all four possible sections of no_index. I also included a test file to cover the new subroutine, so that all my additions are tested. Please let me know if you have any questions or suggestions for further improvement! |
Account for modules in the no_index section of META.yml
Thank you for your great work! |
Thank you for writing this module, it is a nice extension of Test::Kwalitee. I've already successfully used it on Perl::Critic::Git, but I'm having problems retrofitting DBIx::NinjaORM.
DBIx::NinjaORM uses an internal module (t/lib/LocalTest.pm) that I have specified as no_index in META.yml. As a result, it is not visible on MetaCPAN, and the following call in Extra.pm fails:
This is however a valid module used internally by the distribution, and should not be treated as an external use.
With the attached commit, I'm proposing to identify packages in files listed in the no_index section of META.yml, and skipping those when examining external uses. I've tested the change successfully on DBIx::NinjaORM, and I tried to keep your coding style in this patch. Please let me know what you think!