-
Notifications
You must be signed in to change notification settings - Fork 6
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: Discourse 3.3.0 upgrade #278
Conversation
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.
This looks good, great work @alithethird ! I left two comments, they are about tidying up, so you can address them at a later point.
Test coverage for 46f5963
Static code analysis report
|
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.
lgtm
Updating to Discourse v3.3.0
Applicable spec: N/A
Overview
Upgrading the Discourse version from v3.2.0 to v.3.3.0
Added new part to install
imagemagick
from source because the usedimagemagick
is bumped to v7.1.1. Previously it was installed by apt and the version there is 6.9.Rationale
Juju Events Changes
Module Changes
Library Changes
Checklist
src-docs
urgent
,trivial
,complex
)