-
Notifications
You must be signed in to change notification settings - Fork 411
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!: added edition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modules
#491
Conversation
and data_cache_config to mysql and safer_sql modules
@ibhaskar2 you need to change min provider version to 4.74 as these features were part of the release. |
edition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modulesedition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modules
@imrannayer can you review it and merge it? |
@bharathkkb can you plz review? |
edition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modulesedition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modules
/gcbrun |
edition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modulesedition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modules
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.
edition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modulesedition
in mssql, mysql, postgresql, safer_sql and data_cache_config
to mysql and safer_sql modules
@isaurabhuttam - since this is a breaking change can you please add an upgrade doc like this. This should be pretty simple i.e. explaining customers to update version constraints, if they have any for TPG. |
/gcbrun |
/gcbrun |
b/291924307