-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
#24134: Default logic app versions to ~4 #24280
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.
Hi @harshavmb thanks for submitting this. This looks good, but we should deprecate this with a 4.0 feature flag so this won't cause issues for users currently relying on the default value of ~3
. This is an example of a Default value deprecation in another resource here.
c0d73f7
to
f9672bf
Compare
7c1cffd
to
19a42a3
Compare
Hi @catriona-m , Thanks for the review & I amended changes as per the review. I do see all PRs failing |
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.
Thanks @harshavmb LGTM!
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
Default logic app versions to ~4
Fixes #24134