Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
Hi!
Just a few coding standards improvements on vimrc.drush.inc file.
I hope to help.
Thanks.
Comment | File | Size | Author |
---|---|---|---|
#11 | test-code-standard-2841000.png | 50.21 KB | henriquebauer |
#10 | 2841000_10.patch | 9.59 KB | emartoni |
#9 | interdiff-2-7.txt | 1.42 KB | benjifisher |
#7 | coding_standards_for-2841000-7.patch | 8.38 KB | cebasqueira |
#2 | coding_standards-2841000-2.patch | 7.6 KB | gmaltoni |
Comments
Comment #2
gmaltoni CreditAttribution: gmaltoni at CI&T commentedPatch with fixes.
Comment #3
gmaltoni CreditAttribution: gmaltoni at CI&T commentedComment #4
cebasqueira CreditAttribution: cebasqueira at CI&T commentedComment #5
cebasqueira CreditAttribution: cebasqueira at CI&T commented+RTBC
Comment #6
benjifisherThanks for working on this! It mostly looks good, but I do not see why you want to make a long comment line in the last hunk:
I think the only change I would recommend is changing "See" to "@see" on the second line. https://www.drupal.org/node/1354#see
Probably there is a better solution to the problem with newer versions of drush, but that should not be done as part of this issue.
Comment #7
cebasqueira CreditAttribution: cebasqueira at CI&T commentedPatch updated!
Comment #8
gmaltoni CreditAttribution: gmaltoni at CI&T commentedI applied the patch and it worked cleanly.
Comment #9
benjifisherI have uploaded an interdiff showing the changes from the patch in #2 to the one in #7.
Since this issue is all about coding standards, let's get it right.
and
It should be
@see
(lower case) with no ending punctuation. Please look at the documentation I referenced in my previous comment.As long as we are updating these lines, let's also use
https://
in the URLs.Comment #10
emartoni CreditAttribution: emartoni at CI&T commentedHi all!
Fixed the code standard.
Hope its fine now.
Patch attached.
Comment #11
henriquebauer CreditAttribution: henriquebauer commentedLooks good!
I have reviewed the 2841000_10.patch and there are no errors, just warnings as you can see in the image.