Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
1 Mar 2021 at 17:17 UTC
Updated:
16 Mar 2021 at 10:04 UTC
Jump to comment: Most recent
Comments
Comment #2
avpadernoI 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 #3
mokys commentedHello, I've fixed all messages from PAreview on branch 8.x-1.x
Comment #4
bramdriesenComment #5
ysamoylenko commentedHello.
During my module review, I've not found any critical issues.
Unit tests were passed.
The code well commented and maintainable.
No errors were added to the dblog during my testing of the module functionality.
So, I think the module can be marked as passed the Security Advisory coverage.
Comment #6
avpadernoThank you for your contribution! I am going to update your account.
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.