build providers: inject core18 instead of core
Metadata
Current evaluation
Merged. Updates build providers to inject core18 instead of core. Approved by one reviewer with passing CI checks. Changes modify two files with seven additions and seven deletions.
Suggested action: —
No scores available.
Issue body
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged. Updates build providers to inject core18 instead of core. Approved by one reviewer with passing CI checks. Changes modify two files with seven additions and seven deletions. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer sergiusens, the change updates build providers to inject core18 instead of core. The modification was accepted and integrated into the main branch without additional review comments. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged update to build providers configuration, replacing core injection with core18. Authored by sergiusens, the change was integrated directly into the codebase without review comments. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2196 build providers: better injection logic | snapcraft | merged | Merged to master following approval and conflict resolution. Updates build provider injection logic across 12 files (+951/-414 lines). CI passed with 91.85% diff coverage, showing a 0.05% overall decrease. | |
| #3733 providers: user core22 buildd | snapcraft | merged | Merged into snapcraft/7.0 following maintainer approval and passing CI. The update switches provider builds to core22 buildd, modifying two files. Code coverage increased by 0.05%. | |
| #2773 build providers: inject snapd snap for latest feature availability | snapcraft | merged | Merged changes that inject the snapd snap into build providers instead of core, improving feature availability and reducing size. Approved by reviewer, passed CI, and modified 4 files (+59/-12 lines). | |
| #4177 core18: removal | snapcraft | merged | Merged after two approvals and passing CI. Removes core18 support, resolving issue #4165. Changes span 528 files, deleting over 37,000 lines. Temporary core20 test fixes ensured CI passed prior to merge. | |
| #2474 build providers: remove dead code | snapcraft | merged | Merged removal of dead code for legacy cleanbuild support in build providers. Approved by one reviewer, passed CI, and modified three files (+54/-165 lines). | |
| #2720 remote build: switch from core to core18 | snapcraft | merged | Merged change switches remote build base from core to core18, aligning with snapcraft edge. Approved, passed CI, and modified one file. | |
| #2508 Build provider errors | snapcraft | merged | Merged after approval and passing CI checks. Resolved build provider errors across two files (+353/-85 lines). Codecov reported 90.62% diff coverage. | |
| #4532 feat(providers): use core24 buildd daily image for devel | snapcraft | merged | Merged. Switches build-base: devel to the core24 buildd daily image. Approved with passing CI and full coverage. Merged pending external dependency resolution and a craft-providers release. Fixes #4519. | |
| #2940 build providers: remove use of cloud-init | snapcraft | merged | Merged changes removing cloud-init from build providers. Refactored to use _run and _push_file methods, isolating initial setup. Approved by reviewer, passed CI, and updated six files. | |
| #3583 build providers: snapcraft's new base is core20 (CRAFT-544) | snapcraft | merged | Merged into master. Replaced core18 injection with core20 as the new snapcraft base. Approved by one reviewer, passed CI, and achieved 100% diff coverage. |