-
-
Notifications
You must be signed in to change notification settings - Fork 651
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(facade): add merge facade api #3289
Conversation
View Deployment
🥐 🍔 🥓 🥗 🥘 🌯 🍚 🍛 🍖 🍭 🍧 🍝 🥪 🥖 🍪 |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #3289 +/- ##
==========================================
+ Coverage 29.84% 30.09% +0.24%
==========================================
Files 2051 2109 +58
Lines 109053 109719 +666
Branches 23712 23852 +140
==========================================
+ Hits 32548 33015 +467
- Misses 76505 76704 +199 ☔ View full report in Codecov by Sentry. |
It is recommended to implement the following API as defined below.
Add an example (Office Script design) to explain why there is a merge API on Range |
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
close #xxx
Pull Request Checklist