fix: use "devel" bases on build plan
Metadata
Current evaluation
Merged after approval and passing CI. Updates BuildInfos to use devel bases for craft-providers via a new get_effective_base parameter. Removed an obsolete get_build_plan method from Project. Squash-merged.
Suggested action: —
No scores available.
Issue body
For the BuildInfos we want to use "devel" bases, as that is the term that craft-providers expects to create instances based on the development track. We still need to use the regular "base" (e.g. "core24") in other places, so add a customization parameter to get_effective_base() to use either behavior.
- [ ] 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 `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and passing CI. Updates BuildInfos to use devel bases for craft-providers via a new get_effective_base parameter. Removed an obsolete get_build_plan method from Project. Squash-merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged after passing tests and coverage. Updated get_effective_base() to use devel bases for build plans, aligning with craft-providers. Removed an unnecessary Project.get_build_plan() method. Squash-merged. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix to use devel bases for BuildInfos, aligning with craft-providers expectations. Added a parameter to get_effective_base and removed an obsolete get_build_plan method from Project. Squash-merged. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #882 fix(Project): update devel base info | craft-application | merged | Merged following three approvals and successful CI checks. Updates development base information, resolving issue #880 and CRAFT-4719. The change modifies five files, adding 45 lines and removing 18. | |
| #1281 build(deps): update craft-providers to 3.7.1 | rockcraft | merged | Merged update to craft-providers 3.7.1, resolving a stonking failure in build-base: devel and forwarding proxy environment variables to pro methods. Approved and passed CI. | |
| #4505 chore: remove deprecated parameter `build-base` | snapcraft | merged | Merged to remove the deprecated build-base parameter from craft-providers, eliminating a deprecation warning. Approved, passed CI and tests, and modified three files. The fix was already on the main branch. | |
| #4204 providers: remove deprecated `build_base` parameter | snapcraft | merged | Merged into main after approval and CI checks. Removes the deprecated build_base parameter from craft-provider's launched_environment method, aligning with craft-providers 1.11.0+. | |
| #2069 build(deps): update craft-application and craft-providers | charmcraft | merged | Merged craft-application and craft-providers update. Uses unique build environments per platform, fixing a bug where identical build-on/build-for entries shared environments. Reduces multi-base build times. Resolves #2058. | |
| #4462 build(deps): bump craft-providers to 1.20.1 | snapcraft | merged | Merged update bumping craft-providers to 1.20.1. Fixes a bug where improperly set-up base instances cause snapcraft failures by updating base compatibility tags and adding auto-cleanup for inactive instances. Approved and passed CI. |