Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix config 'extends' option processing#6873

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Open
lxa wants to merge1 commit intoCodeception:main
base:main
Choose a base branch
Loading
fromlxa:fix-extends-a2

Conversation

@lxa
Copy link

@lxalxa commentedAug 19, 2025

Fix:extends handling in config files (global + suite)

Fixes#6872

Problem

  • extends was expanded too late in the load process:
    • Defaults were merged into the “inherited” configbeforeextends was resolved.
      → Result: scalar values from parent configs were lost (arrays still merged).
    • params from parent configs were ignored (see also#6799).
  • For suite configs,extends resolution was missing completely for.suite.dist.yml files.
    → Result:extends in suite dist configs was silently ignored.

Fix

  • Movedextends expansion intogetConfFromContents(), which is used for both global and suite config resolution.
  • Now an entirechain of inheritance is expanded immediately when a config is loaded from file.
  • Only after expansion, the normal config merge order applies:
    codeception.dist.yml → codeception.yml → acceptance.suite.dist.yml → acceptance.suite.yml
  • Added check for circularextends references with a clear error message.

Result

  • Scalars and arrays from parent configs are inherited correctly.
  • params from base configs are preserved.
  • extends works in.suite.dist.yml files (not ignored anymore).
  • Behavior is now consistent across global and suite configs.

Notes

  • This matches what the docs currently say aboutextends.
  • If merged, docs might be updated to clarify that the full inheritance chain is expanded first, then the normal config precedence rules apply.

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

extends option in config files not working as expected

1 participant

@lxa

[8]ページ先頭

©2009-2025 Movatter.jp