3.x cleanup - Githubissues

skaut / SkautisNette

Nette extension for Skautis library

BSD 3-Clause "New" or "Revised" License

3 stars 7 forks source link

3.x cleanup #22

Closed xificurk closed 6 years ago

xificurk commented 6 years ago

Drobná dočištění související s odstraněním podpory pro PHP <5.6 a Nette <2.4.
typehinty

fmasa commented 6 years ago

Nestálo by za to zbavit se duplicitních phpdoců v místech, kde je typehint?

fmasa commented 6 years ago

Jen je potřeba brát v potaz, že přidávání typehintů je minor BC break.

Protože ty třídy, kde se tak děje, nejsou final nebo aspoň internal, takže můžou v projektech tyhle třídy dědit a přetěžovat metody, což jim nebude fungovat.

xificurk commented 6 years ago

@fmasa tohle cílí 3.x větev, která zatím nemá release, takže BC breaky nevadí.

fmasa commented 6 years ago

:man_facepalming: Jsem myslel, že jsem na #21