Uh oh!
There was an error while loading. Please reload this page.
all: remove deprecated stuff but keep infra - #572
Conversation
Remove all the deprecated features, but keep the infrastructure and the documentation thereof in place to allow deprecating features in the future. Re: nodejs#570
97f54ad to
fac7d8cComparegabrielschulhof
commented
Oct 24, 2019
Hmmm .. why is travis testing on v6.x? |
cjihrig
commented
Oct 24, 2019
Does getting rid of Line 15 in bcc1d58 |
gabrielschulhof
commented
Oct 24, 2019
@cjihrig no doubt, I just don't know if we have a good reason for keeping it around 🤷 |
gabrielschulhof
commented
Oct 24, 2019
Hmmm ... git blame reveals no good reason. I think I'll drop it. |
mhdawson
commented
Oct 24, 2019
I'm not 100% sure we want to do this. I know we are planning for a SemVer major but the other changes were very unlikely to cause any code to stop compiling/working. This removes functions that may be used. @gabrielschulhof is there a reason we should remove now? |
gabrielschulhof
commented
Oct 24, 2019
@mhdawson only that a semver-major release is an opportunity to drop things. I don't feel strongly about dropping the deprecated stuff. I'm OK with leaving it in until the next major. |
legendecas
commented
Oct 27, 2019
I think #576 should fix the CI failures on Node v6 targets. |
gabrielschulhof
commented
Oct 28, 2019
Closing this for now. We need not be aggressive about removing deprecated stuff until we're really stumbling over it. |
Remove all the deprecated features, but keep the infrastructure and the
documentation thereof in place to allow deprecating features in the
future.