revert: abstract Project class and `_providers_base()`

Metadata

Project: craft-application
Number: #322
Type: pull request
State: merged
Author: mr-cal
Labels: —
Created: 2024-04-24 07:13 PM UTC
Updated: 2024-04-26 04:17 PM UTC
Closed: 2024-04-26 04:17 PM UTC

Current evaluation

Merged #322 (commit e893f1da): partial revert of aeae554/#302, adding a default name@channel Project._providers_base() while keeping devel-base validation. CRAFT-2824.

Resolution: closed resolved

Reason: PR #322 was merged as commit e893f1da ('revert: abstract Project class and `_providers_base()` (#322)'), landing a default name@channel _providers_base() on the Project model so apps no longer subclass it; the follow-up default BuildPlanner landed in #326 (4833c0e9, craft-application 3.0.0).

Evaluated by: mr-cal-bot

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 `tox`? ----- This is a partial revert of 'aeae554 feat!: validate devel bases (#302)'. The base validation from #302 has not been reverted, but there is now a default implementation for `Project._providers_base()` such that applications do not need to subclass the Project model. The default naming convention for bases is `name@channel`. Examples: - `ubuntu@24.04` - `ubuntu@devel` - `almalinux@9` Charmcraft is the only public application that can use this default `_providers_base()`. A default BuildPlanner will come in a subsequent PR. (CRAFT-2824)

Evaluation history

Date	Model	Scores	Action	Summary
2026-10-03 07:59 AM UTC	stealth/space-bunny-alpha	—	closed resolved	Merged #322 (commit e893f1da): partial revert of aeae554/#302, adding a default name@channel Project._providers_base() while keeping devel-base validation. CRAFT-2824.
2026-09-25 04:49 AM UTC	stealth/space-bunny-alpha	—	closed resolved	Merged #322 (e893f1da): partially reverted aeae554/#302, adding a default name@channel _providers_base() while retaining devel-base validation.
2026-09-25 04:49 AM UTC	stealth/space-bunny-alpha	—	closed resolved	Merged PR #322 as e893f1da, partially reverting aeae554: base validation remains, while a default Project._providers_base() removes the need to subclass Project.
2026-07-23 01:11 PM UTC	qwen/qwen3.6-35b-a3b	—	—	Merged partial revert of aeae554, dropping Project class abstraction and _providers_base(). Introduces a default _providers_base() to eliminate mandatory subclassing while retaining base validation. Approved by two reviewers and passed CI.

Update history

No update history recorded yet.

Related work

Caused By: canonical/craft-application#302 (confidence 95%)
aeae554/#302 abstract Project and made _providers_base() abstract; #322 partially reverted it.
Related To: canonical/craft-application#326 (confidence 90%)
Follow-up PR adding Platform model and default BuildPlanner promised in #322, shipped in 3.0.0.
Related To: canonical/craft-application#321 (confidence 70%)
Maintainer comment noted waiting for #321 (hotfix/2.6 merge to main) to land first.

Related issues

No related issues found above the similarity threshold.