-
Notifications
You must be signed in to change notification settings - Fork 756
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
Create 09.06.02.config #3829
Create 09.06.02.config #3829
Conversation
Add webp support
Co-authored-by: Brian Dukes <bdukes@engagesoftware.com>
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, thanks @Tychodewaard!
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.
There is nothing technically wrong here, just putting my "request changes" to make sure nobody merges this before we have decided if the next version is 9.6.2 or 9.7.0.
If 9.7.0 then we must not merge this until we get there. If we do decide we go with 9.6.2 then I think we could just rename the file and get this together with 9.6.2 as it is not any risk IMO
Yeah, if we're doing a 9.6.2 release, let's include this. |
Ok, so let's decide on the next meeting and we can adjust the filename accordingly... |
This not being a breaking change, we can make it happen in 9.6.2
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.
Just taking a note here, I will test an upgrade and a clean install before merging this.
Add webp support
Summary