Deprecate pkg/term and make it an alias for github.com/moby/term#40825
Deprecate pkg/term and make it an alias for github.com/moby/term#40825cpuguy83 merged 3 commits intomoby:masterfrom
Conversation
vendor.conf
Outdated
There was a problem hiding this comment.
Temporarily using my fork until the linked PR is merged
|
Hmm.. ppc64le nodes are failing again with networking issues; This was on this node: https://ci-next.docker.com/public/computer/ppc64le-ubuntu-19/systemInfo - looks like all previous failures were on that node as well; let me disable it to see if it is just that one. |
|
Ah; I see some workers ran out of swap space; are we missing some cleanup step in CI? https://ci-next.docker.com/public/computer/ |
|
I would put "pkg/term: vendor moby/term and make pkg/term an alias" commit first and test with it (to make sure the deprecated stuff works) and only after it add the "remove uses of deprecated pkg/term" commit. Since Github web UI does not show the patches in the order they are applied, please ignore this if commits are already in the order described above. |
|
Also it looks like the package-wide deprecation notice is missing. |
I swapped the last two commits
Does golang support deprecating the package as a whole, or only individual functions? 🤔 Wondering if it adds anything if all functions are already marked "deprecated" |
Thanks! It also need to test it with the commit that starts using the deprecated package (it should at least build, this is a sure way to know the package is working and nothing is amiss).
Yes ("or even a whole package"):
I think it does:
|
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
Signed-off-by: Sebastiaan van Stijn <[email protected]>
|
@kolyshkin updated; added deprecation message for the package |
| // Deprecated: use github.com/moby/term.Winsize | ||
| type Winsize = term.Winsize | ||
|
|
||
| var ( |
There was a problem hiding this comment.
If I remember correctly, for a group like this you can do
// Deprecated: use github.com/moby/term instead
var (
...
)and then you don't have to document/deprecate each individual one.
Not a big deal anyway
kolyshkin
left a comment
There was a problem hiding this comment.
LGTM (I also checked it builds before the last commit)
See moby/moby#40825 Signed-off-by: Yoan Blanc <[email protected]>

opening as draft, pending moby/term#4