Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Reporter:
Created:
10 Jun 2021 at 04:00 UTC
Updated:
21 Dec 2022 at 07:59 UTC
Jump to comment: Most recent
Comments
Comment #2
avpadernoFor these applications, we need a project where, in at least the branch used for the application, most of the commits (if not all the commits) have been done from the user who applies.
The purpose of these applications is reviewing a project to understand what the user who applies understands about writing secure code that follows the Drupal coding standards and correctly uses the Drupal API. When most of the code has been written by another user, this review cannot be easily done.
For this application to continue, we need a project with a branch where most of the commits have been done from dpagini.
Comment #3
avpadernoI am closing this application since there haven't been replies to my previous comment.
Comment #4
dpagini commentedI would like to re-open this issue for consideration of this module, if possible. I know this module is not highly adopted, and therefore there is not a ton of usage for reporting new issues, but I would like to provide any users of this module some comfort that it is covered under the Drupal security policy. I am committed to that.
I offered to take over as a maintainer for this module a few years back, as I've been using it for probably 4+ years now, and the original creator was not as interested in maintaining it, it appeared.
As the maintainer, I'm Acquia triple certified, and I'm also adding a link to my Github profile (which admittedly does not contain much Drupal). I was also a frequent contributor to Acquia BLT.
https://certification.acquia.com/user/7115
https://github.com/dpagini
Comment #5
akshay.singh commentedComment #6
avpadernoSee my previous comment: For this application to continue, we need a project with a branch where most of the commits have been done from dpagini.
Comment #7
dpagini commentedI have a new branch, 2.x, which has been created for D9/10 support, and I have the only commits on that branch. The vague part of this request is... how many commits do you need to see? This feels very subjective at the moment to me.
Comment #8
avpadernoComment #9
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 Slack #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 reviewers.
Comment #10
avpadernoComment #11
dpagini commentedThank you @apaderno! I appreciate your help and guidance!
Comment #12
dpagini commentedWell that was strange, sorry. Those fields seemed to reset on me? I can't change back the "assigned" field in this follow-up though.
Comment #13
avpaderno