Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

revert: abstract Project class and _providers_base() #322

Merged
merged 2 commits into from
Apr 26, 2024

Conversation

mr-cal
Copy link
Contributor

@mr-cal mr-cal commented Apr 24, 2024

  • Have you followed the guidelines for contributing?
  • Have you signed the CLA?
  • 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:

Charmcraft is the only public application that can use this default _providers_base().

A default BuildPlanner will come in a subsequent PR.

(CRAFT-2824)

@mr-cal
Copy link
Contributor Author

mr-cal commented Apr 24, 2024

I'll wait for #321 to land first

@mr-cal mr-cal force-pushed the work/CRAFT-2824-revert-abstract-project branch from 35478bc to 9561a30 Compare April 24, 2024 23:30
This is a partial revert of 'aeae554 feat!: validate devel bases (#302)'.

The base validation is not reverted, but there is now a default
implementation for `Project._providers_base()` such that applications do not
need to subclass the Project model.

Signed-off-by: Callahan Kovacs <[email protected]>
@mr-cal mr-cal force-pushed the work/CRAFT-2824-revert-abstract-project branch from 9561a30 to 9932817 Compare April 25, 2024 17:03
@mr-cal mr-cal requested a review from lengau April 25, 2024 20:20
@lengau lengau merged commit e893f1d into main Apr 26, 2024
6 checks passed
@lengau lengau deleted the work/CRAFT-2824-revert-abstract-project branch April 26, 2024 16:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants