cli: remove deprecated VisitAll, DisableFlagsInUseLine utilities

[thaJeztah]

• relates to cli: deprecate VisitAll, DisableFlagsInUseLine utilities, remove HasCompletionArg #6273

These were deprecated in 6bd8a4b, and are no longer used.

- Human readable description for the release notes

Go SDK: cli: remove deprecated `VisitAll`, `DisableFlagsInUseLine` utilities.

- A picture of a cute animal (not mandatory but encouraged)

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

[vvoland]

We haven't had a release which includes the deprecation: #6273

Do we want to skip that and remove it without any deprecation period?

[austinvazquez]

We haven't had a release which includes the deprecation: #6273

Do we want to skip that and remove it without any deprecation period?

I was thinking we can include them in 28.4 so the "next release" is 29, wdyt?

[thaJeztah]

We haven't had a release which includes the deprecation: #6273
Do we want to skip that and remove it without any deprecation period?

I was thinking we can include them in 28.4 so the "next release" is 29, wdyt?

Yes, we've back ported that patch to the 28.x branch so that we can include it in 28.4;

• [28.x backport] cli: deprecate VisitAll, DisableFlagsInUseLine utilities, remove HasCompletionArg #6276

Slightly fast-tracking these deprecations, although it's quite unlikely someone uses these, other than indirectly through the plugin initialisation which used it internally.