Set command doesn't merge content properly

contributte / neonizer

:wrench: Collection of useful scripts for manipulaton with NEON files. Support composer pre-install / post-install events. Set/get parameters, validate files.

https://contributte.org/packages/contributte/neonizer.html

MIT License

13 stars 4 forks source link

Set command doesn't merge content properly #8

Closed josefbenjac closed 6 years ago

josefbenjac commented 6 years ago

Use case:

I got simple config file parameters.local.neon

parameters:
    recaptcha:
        siteKey: ...
        secretKey: ...

    deploy:
        lastDeployAt: null
        lastCommit: null
        vcsUrl: ...

When I run set command,

composer set-config -- $(pwd)/app/config/parameters.local.neon --deploy.lastDeployAt=$DATE --deploy.lastCommit=$COMMIT

result is

parameters:
    deploy:
        lastDeployAt: ...
        lastCommit: ...

I think problem is in array_merge function in Contributte\Neonizer\TaskSet:80.

f3l1x commented 6 years ago

👍