Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
18 Jun 2019 at 14:23 UTC
Updated:
7 Jul 2019 at 08:39 UTC
Jump to comment: Most recent
Comments
Comment #2
darshanchoudhary commentedComment #3
nickonom commentedThe Coder review module gives only warning:
Comment #4
nickonom commentedForgot to change the status.
Comment #5
darshanchoudhary commentedHi @nickonom,
I have fixed the issues. Can you kindly re-test.
Thanks,
Darshan
Comment #6
avpadernoThank you for applying! I added the Git instructions for non-maintainer users and the PAReview checklist link. Reviewers will check the project and post comments to list what should be changed.
If you haven't done it, yet, please check the PAReview report and fix what needs to be fixed. There could be some false positives; verify that what reported is correct, before making any change.
Comment #7
klausiThanks for your contribution!
info file: "Pfizer Custom" is wrong, this is a public module now.
Otherwise looks good to me, did not see any security issues.
Comment #8
avpadernoThank you for your contribution! I am going to update the project.
These are some recommended readings to help with excellent maintainership:
You can find more contributors chatting on the IRC #drupal-contribute channel. So, come hang out and stay involved.
Thank you, also, for your patience with the review process.
Anyone is welcome to participate in the review process. Please consider reviewing other projects that are pending review. I encourage you to learn more about that process and join the group of reviewers.
I thank all the dedicated reviewers as well.