agentphoenix / tlint Goto Github PK

composer global require tightenco/tlint

This is an opinionated code linter for Tighten flavored code conventions for Laravel and Php.

For example, Laravel has many available ways to pass variables from a controller to a view:

A)

return view('view', ['value' => 'Hello, World!']);

B)

$value = 'Hello, World!';

return view('view', compact('value'));

C)

return view('view')
    ->with('value', 'Hello, World!');

In this case TLint will warn if you are not using the C) method. This example is a sort of "meta layer" of code linting, allowing teams to avoid higher level sticking points of code review / discussions.

TLint also has more immediately useful lints that can supplement your editor/ide such as:

• NoUnusedImports
• TrailingCommasOnArrays
• And many more! (See below for full listing)

For entire project (you must pass the lint command to use other options)

tlint

For individual files and specific directories

tlint lint index.php
tlint lint app

You can also lint only diff files by running the following with unstaged git changes

tlint lint --diff
tlint lint src --diff

Want the output from a file as JSON? (Primarily used for integration with editor plugins)

tlint lint test.php --json

Want to only run a single linter?

tlint --only=UseConfigOverEnv

Linting TestLaravelApp/routes/web.php
============
Lints: 
============
! Prefer `view(...)->with(...)` over `view(..., [...])`.
5 : `    return view('test', ['test' => 'test']);``

TLint Ships with 2 "preset" styles: Laravel & Tighten. The Laravel preset is intended to match the conventions agreed upon by the Laravel framework contributors, while the Tighten preset is intended to match those agree upon by Tighten team members.

The default configuration is "tighten" flavored, but you may change this by adding a tlint.json file to your project's root directory with the following schema:

Note: You may further customize the linters used by adding specific lint names to the "disabled" list (As shown below).

{
  "preset": "laravel",
  "disabled": ["NoInlineVarDocs"]
}

• No leading slashes in namespaces or static calls or instantiations. RemoveLeadingSlashNamespaces
• Fully qualified class name only when it's being used a string (class name). QualifiedNamesOnlyForClassName
• Class "things" should be ordered traits, static constants, statics, constants, public properties, protected properties, private properties, constructor, public methods, protected methods, private methods, other magic methods. ClassThingsOrder
• Sort imports alphabetically AlphabeticalImports
• Trailing commas on arrays TrailingCommasOnArrays
• No parenthesis on empty instantiations NoParensEmptyInstantiations
• Space after sole not operator SpaceAfterSoleNotOperator
• One blank line between class constants / properties of different visibility OneLineBetweenClassVisibilityChanges
• Never use string interpolation without braces NoStringInterpolationWithoutBraces
• Spaces around concat operators, and start additional lines with concat ConcatenationSpacing
• File should end with a new line NewLineAtEndOfFile
• No /** @var ClassName $var */ inline docs NoInlineVarDocs (tighten#108)
• There should be no unused imports NoUnusedImports

• There should be no method visibility in test methods NoMethodVisibilityInTestsTest (tighten#106 (comment))

• Use with over array parameters in view(). ViewWithOverArrayParamaters
• Prefer view(..., [...]) over view(...)->with(...). ArrayParametersOverViewWith
• Don’t use environment variables directly in code; instead, use them in config files and call config vars from code. UseConfigOverEnv
• There should only be rest methods in an otherwise purely restful controller. PureRestControllers
• Controller method order (rest methods follow docs). RestControllersMethodOrder
• Use the simplest request(...) wherever possible. RequestHelperFunctionWherePossible
• Use auth() helper over the Auth facade. UseAuthHelperOverFacade
• Remove method docblocks in migrations. NoDocBlocksForMigrationUpDown
• Import facades (don't use aliases). ImportFacades
• Mailable values (from and subject etc) should be set in build(). MailableMethodsInBuild
• No leading slashes on route paths. NoLeadingSlashesOnRoutePaths
• Apply middleware in routes (not controllers). ApplyMiddlewareInRoutes
• Model method order (relationships > scopes > accessors > mutators > boot). ModelMethodOrder
• There should be no calls to dd() NoDd
• Use request()->validate(...) helper function or extract a FormRequest instead of using $this->validate(...) in controllers RequestValidation
• Blade directive spacing conventions. NoSpaceAfterBladeDirectives, SpaceAfterBladeDirectives
• Spaces around blade rendered content SpacesAroundBladeRenderContent
• Use blade {{ $model }} auto escaping for models, and double quotes via json_encode over @json blade directive: <vue-comp :values='@json($var)'> -> <vue-comp :values="{{ $model }}"> OR <vue-comp :values="{{ json_encode($var) }}"> NoJsonDirective