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

Set command doesn't merge content properly #8

Closed
josefbenjac opened this issue Jul 24, 2018 · 1 comment

Comments

2 participants
@josefbenjac
Copy link
Member

commented Jul 24, 2018

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 added a commit that referenced this issue Jul 27, 2018

@f3l1x f3l1x added this to the v0.3 milestone Jul 27, 2018

@f3l1x f3l1x closed this Jul 27, 2018

@f3l1x

This comment has been minimized.

Copy link
Member

commented Jul 27, 2018

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.