Remove `reportResolvedReference *types.ImageReference` from `NewCopier`

Open flouthoc opened this issue 9 months ago • 3 comments

There is no need of reportResolvedReferece in NewCopier after https://github.com/containers/common/pull/2339 .

I would like to change the public exposed API for NewCopier but it is breaking change and other than podman there are no consumers of it. If maintainers agree then I would like to open a PR.

Mar 26 '25 15:03 flouthoc

@containers/podman-maintainers WDYT ?

Mar 26 '25 15:03 flouthoc

cc @mtrmac for info.

Mar 26 '25 15:03 flouthoc

We have never committed to a stable API in c/common, if you like to break it you can do it as long as you also update the callers.

Mar 26 '25 16:03 Luap99