-
Notifications
You must be signed in to change notification settings - Fork 87
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
doc: Details on adding a Webpack external module for Imperative w/o Secrets SDK #2281
Conversation
require
instead of await import
for Secrets SDK
require
instead of await import
for Secrets SDK
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #2281 +/- ##
=======================================
Coverage 91.24% 91.24%
=======================================
Files 628 628
Lines 17867 17867
Branches 3735 3805 +70
=======================================
Hits 16302 16302
Misses 1564 1564
Partials 1 1 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Trae Yelovich <[email protected]>
620b9eb
to
0b3a57c
Compare
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 @traeok!
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.
Requesting a couple of tiny changes.
Signed-off-by: Trae Yelovich <[email protected]>
Quality Gate passedIssues Measures |
What It Does
Formats the
EXTENDERS.md
document for the Secrets SDKkeyring
module, and adds information on how to add Secrets SDK to yourexternals
section of the Webpack config when needed (rare case).How to Test
Review Checklist
I certify that I have:
Additional Comments