search

keboola / php-db-import

MIT License
4 stars 0 forks source link

Update PHP 7 and cleanup #41

Closed Halama closed 6 years ago

Halama commented 6 years ago

Update před následujícím PR který bude fixovat https://github.com/keboola/php-db-import/issues/39

ondrejhlavacek commented 6 years ago

declare(strict_types=1); nevyžaduje return typy funkcí? ty tam všude chybí

Halama commented 6 years ago

declare(strict_types=1); jenom říká že se typy kontrolují striktně a neprovádí se automatická konverze. A aplikuje se to tam kde jsou typy definované - argumenty funkce nebo return type hint. Ty return type hinty jsem si nechal na konec a pak na ně zapomněl https://github.com/keboola/php-db-import/blob/martin-php-7/phpcs.xml Dobouchám to tam, díky.

Halama commented 6 years ago

Ještě přidávám phpsatan

Halama commented 6 years ago

Přihodil jsem tam ještě ten phpstan, počkal bych až dojedou testy a mergnul bych to.

Halama commented 6 years ago

Zeleno, po obědě bych teda releasnul a pokračoval dál na #42