Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Issue tags:
Reporter:
Created:
13 Nov 2020 at 08:53 UTC
Updated:
3 Dec 2020 at 06:04 UTC
Jump to comment: Most recent
Comments
Comment #2
matroskeenComment #3
matroskeenComment #4
matroskeenComment #5
matroskeenComment #6
matroskeenComment #7
avpadernoComment #8
matroskeenAdding "PAReview: review bonus" after reviewing 4 applications.
Comment #9
kporras07 commentedHi,
I just reviewed this module and the code looks good and also works as expected.
I guess you want to support all versions of D8 and D9 and that's why you have both core and core_version_requirements in the .info.yml file. If that's the case, it's good :) If not, you probably want to remove the core key.
There's no module duplication or licensing issues. Code doesn't seem to be exposing any security issues so I'm marking this as RTBC.
Comment #10
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.
Comment #11
matroskeenThanks!