Closed MichaelGoberling closed 1 year ago
Merging #711 (364ae15) into master (a427f0c) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #711 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 57 57
Lines 3167 3169 +2
Branches 596 596
=========================================
+ Hits 3167 3169 +2
Files Changed | Coverage Δ | |
---|---|---|
src/commands/app/install.js | 100.00% <100.00%> (ø) |
|
src/commands/app/pack.js | 100.00% <100.00%> (ø) |
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more
Description
Currently, the
code-download
(Nowdisable-download
) annotation is added to all actions when an app is packed (aio app pack
) to prevent users who install App Builder apps from downloading action codeThis PR proposes adding the annotation at install time (
aio app install
) insteadRelated Issue
Motivation and Context
Users can technically modify the app zip after
aio app pack
has run. This change guarantees that all installed App Builder apps have action code download disabledHow Has This Been Tested?
npm run test
Screenshots (if appropriate):
Types of changes
Checklist: