SonarQube PHP
  1. SonarQube PHP
  2. SONARPHP-175

PHP CodeSniffer and PHP PMD rules cleanup

    Details

    • Type: Task Task
    • Status: Closed Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 0.4
    • Fix Version/s: 0.6
    • Component/s: None
    • Labels:
      None
    • Number of attachments :
      0

      Description

      It seems that some CodeSniffer rules overlap, and that some descriptions are missing.
      See http://sonar-dev.787459.n3.nabble.com/sonar-dev-PHP-0-5-Tests-before-release-tt3403247.html#a3409299

        Issue Links

          Activity

          Hide
          Fabrice Bellingard added a comment -

          PHPMD rules.xml file: enhanced with missing parameters.

          PHPCS rules.xml file: having a complete rule listing is impossible for the moment as PHP CodeSniffer does not keep track of such a listing. Therefore, PHPCS rules.xml is approximative and misses lots of existing PHPCS rules as long as almost every rule parameters. Gabriele Santini is going to write an FAQ about that. I'll put the link here before resolcing this ticket.

          Show
          Fabrice Bellingard added a comment - PHPMD rules.xml file : enhanced with missing parameters. PHPCS rules.xml file : having a complete rule listing is impossible for the moment as PHP CodeSniffer does not keep track of such a listing. Therefore, PHPCS rules.xml is approximative and misses lots of existing PHPCS rules as long as almost every rule parameters. Gabriele Santini is going to write an FAQ about that. I'll put the link here before resolcing this ticket.
          Hide
          Fabrice Bellingard added a comment -

          The PHPCS rules.xml file has been slightly - which is already a good thing, and a bigger task is planned for PHP 0.7 (SONARPLUGINS-1507).

          Show
          Fabrice Bellingard added a comment - The PHPCS rules.xml file has been slightly - which is already a good thing, and a bigger task is planned for PHP 0.7 ( SONARPLUGINS-1507 ).

            People

            • Assignee:
              Gabriele Santini
              Reporter:
              Fabrice Bellingard
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: