yiisoft / yiisoft/config

Ability for the package to define merge rules for a group

Open
#205 0 comments 1 reaction 0 assignees View on GitHub

Nobody has claimed this yet.

type:feature
Dominant language
PHP
Stars
34
Forks
9
Avg merge
1h 7m
Merged PRs (30d)
1

Description

Proposed new feature or change

In Composer JSON or the PHP config of the package, add an ability to specify the merge strategy for a group:

"config-plugin-options": {
    "source-directory": "config",
    "merge": {
        "widgets-themes": { "recursive-depth": 2 }
    }
}

If there are two packages defining the merge strategy, then throw an exception. It should always be only a single package.

See https://github.com/yiisoft/widget/issues/130#issuecomment-5728823932

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Research direction

Start by locating how Composer JSON and the package's PHP configuration are parsed, then trace the existing group merge handling. Done means a group can specify its merge strategy, and defining that strategy in more than one package raises an exception.

Written by the indexing model from the issue text.

Assessment

Tech stack
php
Domain
tooling
Issue type
Feature
Difficulty
4/5
Estimated time
3-5 days
Activity status
Active
Clarity
Mostly clear
Newbie friendliness
45/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.