search

skaut / SkautisNette

Nette extension for Skautis library
BSD 3-Clause "New" or "Revised" License
3 stars 7 forks source link

Nette\SmartObject instead of Nette\Object #18

Closed jan-stanek closed 6 years ago

jan-stanek commented 6 years ago

stávající verze není kompatibilní s nette/utils 2.5

pravděpodobně rozbije kompatibilitu s nette < 2.4

JindrichPilar commented 6 years ago

Nejsem sice Nettista, ale kdyz nikdo neodpovida :)

K tobe do forku jsem poslal pull request ktery fixne continous integration testy (kdyz ho mergnes).

Odstraneni extends Nette\Object ackoliv to neni na prvni pohled videt je zmena public API. Tedy dle semantickeho verzovani musi byt inkrementovana major verze (SkautisNette 2.x -> 3.x), ovsem Nette ocividne SemVer nepouziva, otazka zda ji musi dodrzovat i tato knihovna.

Zaroven pro Nette 2.4 je minimalni verze PHP 5.6. Pro NetteUtils 2.5 je uz 7.1. Tedy otazka zda neprovest update az na 2.5 i se skokem na nutnost PHP 7.1.

Zaroven je otazka zda udrzovat dve vetve pro ruzne verze Nette a nebo pokracovat vyvoj pouze pro 2.4+

To jsou otazky hlavne na @sinacek

sinacek commented 6 years ago

Udělal bych upgrade až na 3.x s podporou pouze pro PHP 7.1. Stare verze bych dale neudržoval ale nechal bych je dostupne jako posledni v dane vetvi Založil jsem vetev pro 3.x aby se tam dali posilat PR

JindrichPilar commented 6 years ago

Ok, mergnuto do 3.x