[wip] Injection: Generation of informers backed by a dynamic client is opt-in

[dprotaso]

Fixes: #2739

A new flag now exists -include-dynamic-informers to produce the prior dynamic based informers

[knative-prow]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: dprotaso

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:

• OWNERS [dprotaso]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[pierDipi]

I think you need to delete "*_expansion.go" files like: https://github.com/knative/pkg/pull/2740/files#diff-e92a583ae998ed8bf58b1d9fc86498e11542cc490b73567452b26247284a0548

[dprotaso]

/hold

[dprotaso]

Seems like some of the expansion files were generated manually vs. through codegen

[codecov]

Codecov Report

Patch and project coverage have no change.

Comparison is base (300df43) 81.53% compared to head (b86bffc) 81.53%.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #2740   +/-   ##
=======================================
  Coverage   81.53%   81.53%           
=======================================
  Files         163      163           
  Lines       10018    10018           
=======================================
  Hits         8168     8168           
  Misses       1606     1606           
  Partials      244      244           

Impacted Files	Coverage Δ
apis/duck/v1/source_types.go	48.64% <ø> (ø)

... and 2 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[dprotaso]

Going to close this PR in favour of just removing this functionality