-
Notifications
You must be signed in to change notification settings - Fork 85
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
Export new Proxy class from imperative #2205
Conversation
Signed-off-by: Billie Simmons <[email protected]>
Signed-off-by: Billie Simmons <[email protected]>
Signed-off-by: Billie Simmons <[email protected]>
I'm sorry, I am not sure what the CHANGELOG header should be. I don't see an example in any of existing package files and the |
Signed-off-by: Billie Simmons <[email protected]>
Signed-off-by: Billie Simmons <[email protected]>
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! 😋
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 @JillieBeanSim!
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2205 +/- ##
========================================
Coverage 91.15% 91.16%
========================================
Files 638 638
Lines 19133 19134 +1
Branches 4041 3927 -114
========================================
+ Hits 17441 17443 +2
+ Misses 1691 1690 -1
Partials 1 1 ☔ View full report in Codecov by Sentry. |
Release succeeded for the The following packages have been published:
Powered by Octorelease 🚀 |
What It Does
export new Proxy class from imperative
How to Test
Extenders can not import new class using 7.27.0 that introduced the new Proxy support, class needs to be exported from package.
Review Checklist
I certify that I have:
Additional Comments