HomePhabricator

Add a linter rule to detect unnecessary semicolons

Description

Add a linter rule to detect unnecessary semicolons

Summary:
Ref T7409. Adds a rule to detect unnecessary semicolons. The most common scenario I've seen in the wild is the use of semicolons after a class definition:

class MyClass {
  // ...
};

Test Plan: Added unit tests.

Reviewers: Blessed Reviewers, epriestley

Reviewed By: Blessed Reviewers, epriestley

Subscribers: Korvin, epriestley

Maniphest Tasks: T7409

Differential Revision: https://secure.phabricator.com/D12139

Details

Provenance
joshuaspenceAuthored on
joshuaspencePushed on Apr 6 2015, 9:24 PM
Reviewer
Blessed Reviewers
Differential Revision
D12139: Add a linter rule to detect unnecessary semicolons
Parents
rARC92713cf9220c: Remove deprecated `ComprehensiveLintEngine` class
Branches
Unknown
Tags
Unknown
Tasks
T7409: Implement PHPCS lints