logoalt Hacker News

somebehemothtoday at 1:15 AM4 repliesview on HN

I think the constructive criticism is best directed at whatever process you are following. That process allowed a very visible user facing change in a widely used piece of software. How did this change make it to production without some process catching the impact of this change? Was there really no internal discussion from a code review at least? This seems hard for me to believe. I expect more from Microsoft.


Replies

serial_devtoday at 4:43 AM

> Was there really no internal discussion from a code review at least? This seems hard for me to believe.

The outlined story feels unfortunately very believable to me.

Teams need to push out the most number of features, and nobody stops even for a second to think about how a feature might affect other flows or other users not in the feature request.

It might have been quickly reviewed to check if the code does what it needs to do (add the coauthor note).

Do you think reviewers will think about unwanted effects, when they need get back to feeding their own poorly thought out and underspec’d features to their LLMs?

marrickstoday at 1:45 PM

Also, who/what group is pushing for this change internally and what is the opinion of the team implementing it? What is the road map and vision for AI in VSCode?

dmitrivtoday at 7:03 AM

Fair point. We did catch it internally in testing (as we use VS Code for all our work, so some folks did stumble on it), but I think we underestimated the impact and should do a better job at that.

show 2 replies
PunchyHamstertoday at 9:30 AM

It got to production because they wanted it.

> This seems hard for me to believe. I expect more from Microsoft.

Those are some baseless expectations given the entire company's history