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

Enable short array syntax by default #4

Open
weierophinney opened this issue Dec 31, 2019 · 1 comment
Open

Enable short array syntax by default #4

weierophinney opened this issue Dec 31, 2019 · 1 comment
Labels
Enhancement New feature or request

Comments

@weierophinney
Copy link
Contributor

As we support php >= 5.6, we can enable short array syntax in configuration by default.


Originally posted by @snapshotpl at zfcampus/zf-configuration#18

@weierophinney weierophinney added the Enhancement New feature or request label Dec 31, 2019
@weierophinney
Copy link
Contributor Author

Except that it breaks existing applications. Not in a functional way, but in a "why is my configuration completely rewritten on this change?" way.

What will happen is users will have long array syntax in configuration files managed by Apigiliy. When they update to a version of zf-configuration that enables short array by default, and then use Apigility to manipulate their API, these files will switch to short array notation — and something that might have been a single line of changes previously now affects several dozen lines in the file.

As such, until we do a new major version of zf-configuration, we cannot enable short array syntax by default.

We can, however, enable it by default in the skeleton in a new version, and we're planning that for the 1.4 release.


Originally posted by @weierophinney at zfcampus/zf-configuration#18 (comment)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant