This issue is for testing coding standards patches before committing.
| Comment | File | Size | Author |
|---|---|---|---|
| #18 | 3051208-18.patch | 2.03 KB | tr |
| #16 | 3051208-16.patch | 3.5 KB | tr |
| #14 | 3051208-14.patch | 3.65 KB | tr |
| #12 | 3051208-12.patch | 404 bytes | tr |
| #10 | 3051208-10.patch | 605 bytes | tr |
Comments
Comment #2
tr commentedQuote strings in .yml
Comment #4
tr commentedDocumentation comments and @todo
Comment #6
tr commentedWhitespace.
Comment #8
tr commentedTypehints.
Comment #10
tr commentedUse implode() instead of join().
Comment #12
tr commentedUnused use.
Comment #14
tr commentedmimemail.module: Use ' instead of " where appropriate, add @file
Comment #16
tr commentedMore whitespace and line wrapping.
Comment #18
tr commentedMiscellaneous coding standards fixes.
Comment #20
tr commentedI believe all the remaining coding standards warnings are either false positives or do not need to be / should not be fixed. (For example, we don't want to and should not re-wrap all the lines in CHANGELOG.txt just because many are > 80 characters.)
So I'm going to mark this issue as fixed. If there are additional coding standards problems that need to be addressed please open a new issue for each one.