Php upgrade #34
Php upgrade #34
Conversation
lulco
left a comment
lulco
left a comment
There was a problem hiding this comment.
vsade kde je mixed mam otazku: nevieme co to vrati?
vsade kde je v kode \Namespace\Class pouzi use nech to nemame v kode ale hore v use - idealne keby si zapol sniffer do github workflows - mozno takyto https://github.com/lulco/phoenix/blob/master/.github/workflows/codesniffer.yml
| parameters: | ||
| reportUnmatchedIgnoredErrors: false | ||
| ignoreErrors: | ||
| - '#^Method .* is unused\.$#' |
There was a problem hiding this comment.
fuha :)) k tomuto ta co viedlo? ci to kontroluje public metody ktore sa vramci libky nepouziju?
There was a problem hiding this comment.
tu je problem ten, ze na lvl=max mi hadze unused metody, takze som dal ignore. Lenze na lvl=3 necekuje unused metody, tym padom hadze error, ze to pravidlo sa nepouziva. Preto som dal aj reportUnmatchedIgnoredErrors
There was a problem hiding this comment.
nooo ale tak my budeme robit check len na jednom leveli (asi max) tak netreba ignorovat pre level 3
2 participants
No description provided.