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

PHP7 a datové typy #8

Closed
soukicz opened this issue Nov 24, 2016 · 3 comments
Closed

PHP7 a datové typy #8

soukicz opened this issue Nov 24, 2016 · 3 comments

Comments

@soukicz
Copy link
Contributor

soukicz commented Nov 24, 2016

Koukám, že už je dropnuté PHP5. Nechtělo by to ale rovnou i upravit argumenty, ať mají hezky datové typy? Když už se dělá nová major verze...
Klidně pošlu PR, ale moc se mi nechce drbat s code style :))

@mhujer
Copy link
Owner

mhujer commented Nov 24, 2016

Já už tu knihovnu nepoužívám na ničem běžícím, takže jsem se je bál doplnit, aby se to pak někde nevysypalo. Navrhuju, že to udělám v branchi, ty to otestuješ na reálných datech a pak to dáme do masteru, což?

@soukicz
Copy link
Contributor Author

soukicz commented Nov 24, 2016

Klidně. Já si právě říkal, že major verze je pro to ideální příležitost.

@mhujer mhujer mentioned this issue Dec 9, 2016
6 tasks
@mhujer
Copy link
Owner

mhujer commented Dec 9, 2016

Nakonec jsem to vzal ještě o něco šířeji - #9

@mhujer mhujer closed this as completed in #9 Aug 4, 2017
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

No branches or pull requests

2 participants