Refines the create template function APIs#866
Conversation
|
@matt-gadd is this the kind of change you were thinking for the issue? This still requires docs to be updated and might be nice to provide a shim for the removed APIs to make it non breaking with a deprecated warning? |
|
This pull request is automatically built and testable in CodeSandbox. To see build info of the built libraries, click here or the icon next to each commit SHA. Latest deployment of this branch, based on commit e5e7052:
|
Codecov Report
@@ Coverage Diff @@
## master #866 +/- ##
=======================================
Coverage 94.72% 94.72%
=======================================
Files 127 127
Lines 8095 8096 +1
Branches 1889 1890 +1
=======================================
+ Hits 7668 7669 +1
Misses 427 427
Continue to review full report at Codecov.
|
Type: feature
The following has been addressed in the PR:
prettieras per the readme code style guidelinesDescription:
Merges createResourceTemplate, createResourceTemplateWithInit and createMemoryResourceTemplate in a single
createResourceTemplatefunction that supports creating templates with and without init options and defaults to the old "memory template" which is not going to be called the "default template".Resolves #864