Closed florind-ens closed 2 years ago
Merging #262 (d44aa9d) into master (5e42bf2) will not change coverage. The diff coverage is
100.00%
.
@@ Coverage Diff @@
## master #262 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 65 65
Lines 1675 1698 +23
Branches 317 322 +5
=========================================
+ Hits 1675 1698 +23
Impacted Files | Coverage Δ | |
---|---|---|
src/commands/runtime/trigger/list.js | 100.00% <ø> (ø) |
|
src/commands/runtime/package/delete.js | 100.00% <100.00%> (ø) |
Continue to review full report at Codecov.
Legend - Click here to learn more
Δ = absolute <relative> (impact)
,ø = not affected
,? = missing data
Powered by Codecov. Last update 5e42bf2...d44aa9d. Read the comment docs.
Tested, works well. Aside from the help text change, LGTM.
Added support for non-empty packages deletion.
Description
Usage:
aio runtime package delete packageName --recursive
. Will delete the package namedpackageName
, will delete actions, rules & triggers associated with the actions.Related Issue
Closes #159
Motivation and Context
How Has This Been Tested?
unit tests. Steps:
aio
project, execute:aio runtime package list
to retrieve a list of the packages. Identify the package you just deployed.aio runtime package delete packageName
should return an error.aio runtime package delete packageName --recursive
should delete the package and the action & rule & trigger configured.Screenshots (if appropriate):
Types of changes
Checklist: