search

Gert-dev / php-ide-serenata

Atom IDE package that integrates the Serenata server to provide PHP code assistance
https://serenata.gitlab.io/
Other
274 stars 19 forks source link

An int must not be an integer in the document comment #454

Closed linjialiang closed 5 years ago

linjialiang commented 5 years ago

snipaste_2018-10-30_23-20-40

Gert-dev commented 5 years ago

This will not be an issue anymore in v5.0 as linting will be removed there.

Also, you may want to switch integer to int in the docblock anyway since integer is a legacy name that predates PHP 7 type hints and several other tools - such as PHPStan I believe - also don't like them anymore.

linjialiang commented 5 years ago

The next major release will no longer support linting !

If you need to install atom-ide-ui linter php-ide-serenata in the future, is this a problem for compatibility ?

Gert-dev commented 5 years ago

I slightly misphrased my answer above - linting will not be entirely removed, but semantic linting will be removed. In other words, syntax errors will still be highlighted.

To answer your question: either linter or the linter provided by atom-ide-ui should currently be fine as they are both compatible with the same API.

linjialiang commented 5 years ago

ok