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

breaking: Switch to ST124 "shorthand notation" syntax in charmcraft.yaml #258

Draft
wants to merge 7 commits into
base: charmcraft-3
Choose a base branch
from

Conversation

carlcsaposs-canonical
Copy link
Contributor

Enables building & releasing multi-base charms with 24.04 in a single charmcraft.yaml and git branch

Integration testing is not supported on multiple bases—it is currently only supported on 22.04

Breaking changes

  • ST124 "shorthand notation" syntax required in charmcraft.yaml
  • pytest-operator-cache (which overrides ops_test.build_charm from pytest-operator) now assumes 22.04 base

Deprecation notice

  • pytest-operator-cache is deprecated & will be removed in a future release

Enables building & releasing multi-base charms with 24.04 in a single charmcraft.yaml and git branch

Integration testing is not supported on multiple bases—it is currently only supported on 22.04
Adds support for core24 `platforms` in snapcraft.yaml
@carlcsaposs-canonical
Copy link
Contributor Author

Will not be merged until coordinated charmcraft 3 migration (https://chat.canonical.com/canonical/pl/wekd7r4eqtdn7mxhgxgej8zzsr)

Depends on #258

Comment on lines +9 to +10
# TODO: use permalink
warnings.warn(
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will address right before merging (when version number for permalink can be predicted)

@carlcsaposs-canonical carlcsaposs-canonical changed the title breaking: Support ST124 shorthand notation syntax in charmcraft.yaml breaking: Switch to ST124 "shorthand notation" syntax in charmcraft.yaml Dec 19, 2024
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.

1 participant