search

keboola / php-utils

A set of utility functions
MIT License
0 stars 1 forks source link

refactor: move implementation to shared library #3

Closed odinuv closed 5 years ago

odinuv commented 5 years ago

bwd compatible way to gradually get rid of this repository

ondrejhlavacek commented 5 years ago

co je na tomhle repozitáři špatnýho?

ondrejhlavacek commented 5 years ago

s review počkám na merge a release https://github.com/keboola/sanitizer/pull/1

ondrejhlavacek commented 5 years ago

a pokud to chceš pomalu vyhladit, proč to děláš zpětně kompatibilní? co nová major verze, kde ty vyjmutý fce budou chybět? nebo mi něco uniká?

odinuv commented 5 years ago

@ondrejhlavacek na php-utils je spatny uplne vsechno :) https://keboola.slack.com/archives/C9FEAHW7N/p1540226489000100

momentalne treba to, ze abych do output mappingu dostal webalizer nazvu sloupce, tak sebou potahnu jsonlinter, wtf

delam to zpetne kompatibilni, protoze jinak bych nekde jinde vygeneroval zarovku. Pokud budes nekde potrebovat updatovat neco nesouvisejiciho, co pouziva php-utils, tak budes rad, ze to je zpetne kompatibilni :).

ondrejhlavacek commented 5 years ago

Pokud budes nekde potrebovat updatovat neco nesouvisejiciho, co pouziva php-utils, tak budes rad, ze to je zpetne kompatibilni :).

A stane se to?

odinuv commented 5 years ago

A stane se to?

to nevim :)