1
100%
master: 100%

Ran 11 Jun 2026 05:06PM UTC

Files 20

Run time 1s

Badge

Embed ▾

Committed 11 Jun 2026 04:28PM UTC coverage: 100.0%. Remained the same

Job # 782e86a8-560e-4fe3-84c9-63705f0e285d.1

Build Type

Pull #1318

circleci

Committed by

omry

Commit Message

Fix nested readonly structured config merges

OmegaConf.merge() and OmegaConf.unsafe_merge() construct their results by mutating an internal merge target. Nested frozen structured configs can carry explicit readonly flags on destination containers, so recursive merge writes into those containers could raise ReadonlyConfigError even though public merge_with() is the operation that should reject readonly mutation.

Thread a private _allow_readonly_target flag through the merge internals so non-public merge construction can temporarily relax a destination container's local readonly flag, then restore the saved flag after the merge. Keep public merge_with() semantics unchanged.

Add regression coverage for OmegaConf.merge() and OmegaConf.unsafe_merge() with nested frozen structured configs, their input mutation contracts, and nested readonly flag restoration.

Fixes #1102

Pull Request Pull Request #1318: Fix nested readonly structured config merges

Coverage Stats

3819 of 3819 relevant lines covered (100.0%)

1.0 hits per line

omry / omegaconf / 782e86a8-560e-4fe3-84c9-63705f0e285d / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 782e86a8-560e-4fe3-84c9-63705f0e285d.1

omry / omegaconf / 782e86a8-560e-4fe3-84c9-63705f0e285d / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 782e86a8-560e-4fe3-84c9-63705f0e285d.1

omry / omegaconf / 782e86a8-560e-4fe3-84c9-63705f0e285d / 1
100%
master: 100%

README BADGES
x