3 ms·
This is fine to me, tbh. It's really hard to understand the context of these comments. If you're not reading the code, it makes little sense to see the comments
by bastawhiz 4y ago
This is fine to me, tbh. It's really hard to understand the context of these comments. If you're not reading the code, it makes little sense to see the comments on that code. You can't even review a PR from that view, so having them in the timeline doesn't actually help prevent mistakes. For long PRs, the timeline gets very cluttered with comments and becomes almost unusable.
- swayvil 4y ago>It's really hard to understand the context of these comments. You, sir, have been caught. At the time of this writing there is only 1 other comment.
- kemayo 4y agoAmusingly enough, unless this is your joke whooshing over my head, in this case you're the one not understanding the context of a comment. OP is talking about the same comments the article is talking about -- those on commits on GitHub.
- swayvil 4y agofair point.
- bastawhiz 4y agoI'm talking about the comments that GitHub is removing from the timelines.