Skip to content

Commit

Permalink
Reword inspections for consistency
Browse files Browse the repository at this point in the history
  • Loading branch information
drjayvee committed Oct 20, 2024
1 parent 6419e34 commit df8a862
Showing 1 changed file with 7 additions and 7 deletions.
14 changes: 7 additions & 7 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -103,10 +103,10 @@ Note that most of these could also be analyzed by PHPStan if it could properly u
they are rendered. This is the aim of a similar project: [TwigStan](https://github.com/twigstan/twigstan).

## Typed variables
*Invalid type declared (e.g., `{% types {i: 'nit'} %}`)
* ✅ Context contains variable (unless defined as optional)
* ✅ Runtime variable is null when type is not nullable
* ✅ Runtime type doesn't match declaration
*Declared types is invalid (e.g., `{% types {i: 'nit'} %}`)
* ✅ Context does not contain non-optional variable
* ✅ Runtime: non-nullable variable is null
* ✅ Runtime: variable does not match type
* ⌛ Invalid object property or method (e.g., `{{ user.nmae }}`)
* ⌛ Undocumented context variable (i.e., missing in `{% types %}`)
* ⌛ Use of short-hand form (e.g., `{{ user.admin }}` instead of `isAdmin`) [Notice]
Expand All @@ -128,9 +128,9 @@ While Twig considers all macro arguments optional (and provides `null` as a defa
no explicit default value as required.

* ⌛ Arguments not declared using `types`
*Use of undefined variables (arguments, `{% set %}`, etc)
*Calls with *too many* arguments (except if `varargs` is used)
*Calls with *too few* arguments
*Undefined variable used (arguments, `{% set %}`, etc)
*Call with *too many* arguments (except if `varargs` is used)
*Call with *too few* arguments
* ✅ Required argument declared after optional
* ⌛ Type mismatch in macro call

Expand Down

0 comments on commit df8a862

Please sign in to comment.