Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Composer: define specific dependencies verions #13

Closed
wants to merge 1 commit into from
Closed

Composer: define specific dependencies verions #13

wants to merge 1 commit into from

Conversation

f3l1x
Copy link
Contributor

@f3l1x f3l1x commented Feb 12, 2016

@uestla

Mozna by nebylo spatne mit definovane konkretni verze. Tezko rict, jestli treba Nette\Database se nebude od verze 2.4 vs 2.3 nejak lisit.

Take jsem moc nezkousel treba verzi 2.0, ale tam uz to nechci menit.

@f3l1x
Copy link
Contributor Author

f3l1x commented Mar 5, 2016

@uestla Co si o tom myslis?

@uestla
Copy link
Owner

uestla commented Mar 5, 2016

@f3l1x Ohledně verzí se tak nějak řídím tím, co najdu v composer.json v nette/sandbox.

Teď jsem se (konečně) dokopal podívat se, jaký je rozdíl mezi tildou ~ a stříškou ^.

Nevím, jak je na tom Nette se sémantikou verzování, ale stříška by měla zajistit, že se bude updatovat do další major verze (vyjma), čili bez BC breaků.

Jako konkluzi bych viděl zůstat u stříšky (když už ji má i oficiální Nette balík) a z tohodle pullu vyňal pouze nastavení verzí u knihoven potřebných pro dev - i když si nejsem jist, jestli když composer hledá průnik verzí uspokojujících všechny knihovny, jestli ta hvězdička vadí...

Jak to vidíš ty?

@f3l1x
Copy link
Contributor Author

f3l1x commented Mar 10, 2016

Ja jsem ^ nikdy nepouzil, takze jsem si to musel take nastudovat.

Nicmen, aktualne tam mas nette/database: 2.*. Tzn, ze to vezme 2.0, 2.1, 2.2, 2.3 a 2.4 a dal.

Nejsem si jisty, ze 2.4 bude kompatibilni s 2.3.

Mozna se bavime o tom stejnem. :)

@uestla
Copy link
Owner

uestla commented Mar 15, 2016

Teď je tam právě ^2.3 stejně jako v nette/sandbox. Podle sémantického verzování by to mělo být kompatibilní i s vyššími 2.* verzemi...

Možná by bylo rozumnější ty verze v tuhle chvíli omezit více a případně až v budoucnu je rozšiřovat na vyšší a být v tom benevolentnější. Je to celkem dilema...

@f3l1x
Copy link
Contributor Author

f3l1x commented Mar 16, 2016

@uestla Aha, uz asi tusim kde je komunikacni sum.

Ano, podle verzovani by 2.1 a 2.3 nemela obsahovat BC break. Nicmene, nette\database 2.0 a 2.3 rozhodne nejsou kompatibilni.

Nicmene, jsem si blbe precetl, ze 2.* a ^2.3 a ~2.3 neni to same. To se omlouvam.

Tak asi staci definovat jenom ty dev zavilosti nejak konkretneji. Ale to uz necham na tobe.

:)

@uestla
Copy link
Owner

uestla commented Jul 19, 2016

Dev závislosti definovány dnes v b2574cd :)

Díky.

@uestla uestla closed this Jul 19, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants