[Code Review] Reminder about links in code & commit messages

It sounds to me like most of the contention is around using these links in commit messages, and nobody is arguing to keep the links in source or test files themselves. If that’s accurate, I think I can hold my nose with commit messages because of the tooling argument. We already allow Fixes <link> and Differential Revision: <link> in commit messages (because of tooling and because of community benefit to linking information together), so allowing other accoutrements to help tooling seems reasonable enough so long as it doesn’t start to overwhelm the useful information in the commit message.

7 Likes