Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
5 Jun 2019 at 06:32 UTC
Updated:
23 Oct 2019 at 21:58 UTC
Jump to comment: Most recent
Comments
Comment #2
maheshkp92Comment #3
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 #4
avpadernoThe code doesn't follow the Drupal coding standards in some points.
Comment #5
klausi@maheshkp92 the project page is empty. Can you fill in all infos like https://www.drupal.org/node/997024 ?
Coding standards are not application blockers and I don't see any security issues, so this should be good to go!
Comment #6
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.
Comment #7
maheshkp92I have clear all the required "Pareview" issues.
Comment #8
avpadernoThe application has been already approved, but given the code used for the application, I edited the project, not the account.
Comment #10
avpaderno