Hacker News new | ask | show | jobs
by xahrepap 182 days ago
> That would probably not trigger anyone’s midnight pager, but it would make it clear that relying on the deprecated functionality is a bug lurking in the code.

How do you know? This is a wild assertion. This idea is terrible. I thought it was common knowledge that difficult to reproduce, seemingly random bugs are much more difficult to find and fix than compiler errors.

If you're ready to break your api, break your api. Don't play games with me. If more people actually removed deprecated APIs in a timely manner, then people will start taking it more seriously.
3 comments
Certhas 182 days ago
Last paragraph of the article:

> In case the sarcasm isn’t clear, it’s better to leave the warts. But it is also worthwhile to recognise that in terms of effectiveness for driving system change, signage and warnings are on the bottom of the tier list. We should not be surprised when they don’t work.

link
xahrepap 182 days ago
Yeah, totally a woosh moment for me. Read all the way up to the `* * *`. That's on me :)
link
chrismorgan 182 days ago
That last bit was added to the article after your comment, as the author realised the sarcasm had been too subtle for most people to catch.
link
layer8 182 days ago
Most HN visitors won’t read to the last paragraph, so it’s a good thing to emphasize.
link
xahrepap 182 days ago
I thought I had read it. :) I thought the three `* * *` at the bottom was indicating I was about to start reading suggestions for the next article. So definitely a "Woosh" moment for me :D
link
pavel_lishin 182 days ago
Yeah, I agree - this sort of intermittent failure could be incredibly hard to track down, and will absolutely fuck with people's faith in their CI systems as well - a flappy test is the absolute worst kind of test.
link
matthewkayin 182 days ago
I agree, maintainers should just break the API if they're going to do it.

At the same time, it's crazy that urllib (the library mentioned in the article), broke their API on a minor version. Python packaging documentation[1] provides the sensible guideline that API breaks should be on major versions.

[1] https://packaging.python.org/en/latest/discussions/versionin...

link