pnp/cli-microsoft365

Add `--force` option to `spo site appcatalog remove` command

Closed this issue ยท 6 comments

Noticed that the command spo site appcatalog remove doesn't have a force parameter. Destructive commands should ask for confirmation.

Let's add option:

Option Description
-f, --force Don't prompt for confirmation

TBH since this is a typical thing we have I wonder if we shouldn't consider this as bug and resolve this before v8.
@pnp/cli-for-microsoft-365-maintainers what you think?

TBH since this is a typical thing we have I wonder if we shouldn't consider this as bug and resolve this before v8. @pnp/cli-for-microsoft-365-maintainers what you think?

Scripts that currently are using this command will break, so it is a breaking change imo.

Scripts that currently are using this command will break, so it is a breaking change imo.

That is exactly what I was thinking. Existing scripts will fail if we don't treat this as a breaking change. You can use this command just fine as it's documented right now.

We have another example in our v5.0.0 release: https://pnp.github.io/cli-microsoft365/v5-upgrade-guidance#extended-entra-oauth2grant-remove-with-confirm

Since we have v8 branch we may now start working on it!
Let's get this ๐Ÿšข
and Let's put CLI to the ๐Ÿš€๐ŸŒ™

Can I work on it?

merged to v9