Opened 14 years ago
Closed 13 years ago
#119 closed task (wontfix)
PHPCodeSniffer report is different on Linux and Windows
Reported by: | adube | Owned by: | |
---|---|---|---|
Priority: | trivial | Milestone: | 1.2.0 |
Component: | packaging/scripts/config | Version: | trunk |
Keywords: | phpcodesniffer | Cc: |
Description
When running PHPCodeSniffer on the core source, we get different results on Windows and Linux. Here are some test results made on r840 :
- Linux 10.04 and 8.04, with either the system version (1.2.2) or the one included in GeoPrisma lib (1.2.0a1) by using the 'test' button on the website (locally) :
- 6 errors
- 2 warnings
- on www.geoprisma.org :
- 234 errors
- 2 warnings
The only error that is repeated 228 times is the following : There must be a single space between the closing parenthesis and the openning brace of a multi-line IF statement, found 3 spaces on Sniff Borealis.ControlStructures.MultiLineConditionSniff
On Linux, there IS actually only 2 spaces, but on Windows ( tested with version 7, with Notepad++ ) there is an extra space.
Change History (3)
comment:1 by , 13 years ago
Milestone: | → Future |
---|
comment:2 by , 13 years ago
Milestone: | Future → 1.2.0 |
---|---|
Version: | → trunk |
comment:3 by , 13 years ago
Resolution: | → wontfix |
---|---|
Status: | new → closed |
Note:
See TracTickets
for help on using tickets.
Shouldn't matter anymore with #356.