// DNM Github Action
Leave comments for Future You prefixed with
DNM (DO NOT MERGE) to prevent accidental merges:
// DNM - Switch to finalized PROD URL
Vercel preview deployments are amazing. So amazing, that it’s not crazy to put up a PR to get something you can share for feedback, even if you’re not quite done. While better engineers would never dream of clicking “Merge” before doing a final review, sometimes it’s just too easy.
One approach to prevent this is to leave clues for Future You in your code:
<!-- DNM - UPDATE WITH FINAL CHANGES FROM LEGAL -->
This Github Action serves as your second pair of eyes, failing anytime it detects the string DNM (or whatever you choose) in your PR, and then annotating the lines you still need to address:
Simple, but effective.