feat: add override-overlay
Metadata
Current evaluation
Merged the override-overlay feature. Approved by two reviewers with 180 additions and 35 deletions across 10 files. Passed required CI checks and was successfully integrated into the codebase.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the guidelines for contributing?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint && make test`?
- [ ] Have you added an entry to the changelog (`docs/reference/changelog.rst`)?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged the override-overlay feature. Approved by two reviewers with 180 additions and 35 deletions across 10 files. Passed required CI checks and was successfully integrated into the codebase. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the override-overlay feature following successful lint and test execution. Maintainer smethnani integrated the changes without updating the changelog. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1473 docs: add override-overlay | craft-parts | merged | Merged documentation update adding override-overlay. Approved by three reviewers and passed CI checks despite two failing edge tests. Changes span six files with 20 additions and 8 deletions. | |
| #299 feat: enable override-overlay | imagecraft | merged | Merged following approval from two reviewers and passing CI checks. Updated craft-parts to ~2.30 to enable override-overlay support and added a corresponding spread test. | |
| #110 feat: Support overlay | imagecraft | merged | Merged to enable the overlay feature, provide a bare base, and add spread tests. Approved by two reviewers with passing CI checks. Resolves #98 and CRAFT-4269 across 18 files. | |
| #1165 feat: enable override-overlay | rockcraft | merged | Merged. Updates craft-parts to enable the override-overlay key, replaces overlay-script in tests, and updates documentation. Reviewers approved the changes and confirmed upgrading sphinx-terminal resolves linting errors. All required checks passed before merge. | |
| #1035 feat: reconcile overlays and partition features | craft-parts | merged | Merged. Implements simultaneous overlay and partition feature support, resolving issue #1028. Approved by three reviewers with passing CI. Further tests and refactoring will follow in a subsequent PR. | |
| #1415 feat: add override-overlay | craft-parts | merged | Merged override-overlay, enabling scriptlets to run in a chroot. Adds grammar support, mutual exclusivity with overlay-script, chroot mountpoints, layer hashing, and tests. Approved and passed CI. | |
| #592 fix: delay enabling the overlay feature | rockcraft | merged | Merged fix for #585 that delays overlay feature enablement from module import to Project object creation. Approved by two reviewers, passed CI, and updated six files. | |
| #322 docs: enable inclusion of overlay document | rockcraft | merged | Merged documentation changes enabling overlay document inclusion. Approved by three reviewers, passed CI checks, and implemented via a single-line configuration update. | |
| #4628 feat: allow for overriding project asset handling | snapcraft | merged | Merged. Enables overriding project asset handling by consolidating executable settings and adding tests for hook interactions. Approved by two reviewers, passed CI checks, and maintained test coverage. | |
| #407 features: reorganize overlay feature tests | craft-parts | merged | Merged changes reorganizing overlay feature tests to run with the feature enabled and disabled. Fixes part environment setup to only set CRAFT_OVERLAY when active. Approved by reviewer and passed CI. |