Closed (fixed)
Project:
Drupal.org security advisory coverage applications
Component:
module
Priority:
Normal
Category:
Task
Assigned:
Unassigned
Reporter:
Created:
7 Jul 2020 at 13:28 UTC
Updated:
28 Mar 2021 at 12:22 UTC
Jump to comment: Most recent
Comments
Comment #2
jimmy_sebastian commentedHi @gaurav.kapoor,
I see that phpcs shows few errors.
FILE: /home/lenovo/Public/sri8/modules/tweetext/tweetext.module
---------------------------------------------------------------------------
FOUND 5 ERRORS AFFECTING 4 LINES
---------------------------------------------------------------------------
11 | ERROR | [x] Line indented incorrectly; expected 0 spaces, found 2
12 | ERROR | [x] Expected 3 space(s) before asterisk; 1 found
13 | ERROR | [x] Expected 3 space(s) before asterisk; 1 found
23 | ERROR | [x] Expected 1 space between double arrow and "Url"; 2 found
23 | ERROR | [x] Expected 1 space after "=>"; 2 found
---------------------------------------------------------------------------
PHPCBF CAN FIX THE 5 MARKED SNIFF VIOLATIONS AUTOMATICALLY
---------------------------------------------------------------------------
Time: 1.58 secs; Memory: 8MB
Comment #3
harishh commentedComment #4
gaurav.kapoor commented@jimmy_sebastian I have updated the module and fixed all the issues identified by PHPCS.
Comment #5
harishh commentedHi @gaurav.kapoor,
I have reviewed and not found any errors. Also the functionalities are working fine. looks good.
Comment #6
gaurav.kapoor commentedThanks, @harishh and @jimmy_sebastian.
Comment #7
avpadernoComment #8
phthlaap commentedThanks for your contribution, I haven't seen any security issue related to the code.
Comment #9
gaurav.kapoor commentedThanks @phthlaap
Comment #10
klausimanual review:
This module on its own does not contain a lot of code, but given that gaurav.kapoor has lots of commits on other projects and experience with maintaining projects we can do a full promotion here.
Thanks for your contribution, Gaurav!
I updated your account so you can opt into security advisory coverage now.
Here are some recommended readings to help with excellent maintainership:
You can find lots more contributors chatting on Slack or IRC in #drupal-contribute. So, come hang out and stay involved!
Thanks, 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.
Thanks to the dedicated reviewer(s) as well.
Comment #11
gaurav.kapoor commentedThanks @klausi. I will check the URL's shared by you.
Comment #13
avpaderno