fix: ACNA-1807 - remove deprecated flags in the commands

[shazron]

There have been deprecated flags in the app plugin. e.g. --skip-??? that already have other flag equivalents.
Removal of these flags is a breaking change, thus a major version bump to v10.

For the app deploy command I didn't change skip-deploy to deploy with allowNo:true because it didn't make sense at all, and removed any tests for skip-deploy.

How Has This Been Tested?

manual testing of app:

• deploy
• undeploy
• build
• run

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• I have signed the Adobe Open Source CLA.
• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

[codecov]

Codecov Report

Merging #586 (112248b) into master (488b207) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##            master      #586   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           53        53           
  Lines         2804      2796    -8     
  Branches       528       520    -8     
=========================================
- Hits          2804      2796    -8     

Impacted Files	Coverage Δ
src/commands/app/undeploy.js	100.00% <ø> (ø)
src/commands/app/build.js	100.00% <100.00%> (ø)
src/commands/app/deploy.js	100.00% <100.00%> (ø)
src/commands/app/run.js	100.00% <100.00%> (ø)

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[sandeep-paliwal]

@shazron we use one of the skip flags in our Github action( see https://github.com/adobe/aio-apps-action/blob/master/index.js#L29 ). We should update that code as well to use alternatives and have those actions published and referred to in CI/CD templates. Maybe needs to be tracked as separate issue.

[shazron]

@shazron we use one of the skip flags in our Github action( see https://github.com/adobe/aio-apps-action/blob/master/index.js#L29 ). We should update that code as well to use alternatives and have those actions published and referred to in CI/CD templates. Maybe needs to be tracked as separate issue.

Thanks Sandeep, changing it is easy enough in the action and generator-aio-app add-ci generator. I can add those PRs.
However I saw https://github.com/marketplace/actions/aio-apps is v1 while https://github.com/adobe/aio-apps-action is v2.0.1 - is there a reason v2 is not published to the marketplace

[purplecabbage]

lgtm!
removing these is not a breaking change, deprecating them was ...

[shazron]

@sandeep-paliwal I filed:

1. fix: update --skip-build (deprecated) to use --no-build aio-apps-action#15 (for a newer 2.0.2 release, see next PR)
2. fix: update testing under node-16, update adobe/aio-apps-action version to 2.0.2 generator-aio-app#202