-
Notifications
You must be signed in to change notification settings - Fork 73
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
feat: django-framework extension #1655
Conversation
This pull request introduces an experimental Flask extension for the Juju 12-factor charm ecosystem in charmcraft.
Introduce a new profile called `flask-framework` to the charmcraft init command to create basic projects for 12-factor Flask charms
fixes canonical#1369 Multi-platform build: https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca - [x] [armhf](https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca/+build/2296258) - [x] [arm64](https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca/+build/2296257) - [x] [ppc64el](https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca/+build/2296256) ([buildlog](https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca/+build/2296256/+files/buildlog_snap_ubuntu_jammy_ppc64el_snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca_BUILDING.txt.gz)) - [x] [s390x](https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca/+build/2296255) ([buildlog](https://launchpad.net/~lengau/+snap/snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca/+build/2296255/+files/buildlog_snap_ubuntu_jammy_s390x_snapcraft-charmcraft-084dfcb7321972fd248f5d75baf555ca_BUILDING.txt.gz))
Add the `django-framework` extension, which is similar to the `flask-framework` but specifically designed for the Django framework. The `flask-framework` has been refactored in hope to be more aligned with the extension system in charmcraft. Due to the fact that we are still waiting for some changes in other libraries to land, one spread test might fail. Please review the current content, and I will update the pull request once the changes in other libraries are merged. --------- Co-authored-by: Alex Lowe <[email protected]>
Given |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self-review done
Ok, I think I added |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
self review done
Head branch was pushed to by a user without write access
9e5312d
to
39b5e26
Compare
As discussed with @sergiusens we are proposing to keep the django extension in
latest/edge
in experimental and not pick it for candidate releases until the extension is no longer experimental.