Hacker News new | past | comments | ask | show | jobs | submit login

I worked on a codebase that had a special logging function with a name like NoPushLog that was just a direct wrapper of the base log function. A githook checked that this string was not in pushed code.

This solves many of the concerns raised in this thread about readability, automation, avoiding typos in the magic string.

The tricky thing you have to solve is how to push the code that defines the custom logging function, but there are solutions.




It doesn’t allow you to put that in comments, config files or other languages though like a plain text string.


Why wouldn’t it? Sounds like the git hook is probably just grep…


Then why have a function?


Because of you made a typo, you're more likely to notice than in a comment or string, because it will fail, even at build time if you're using a compiled language


Right, but the whole point of DO NOT SUBMIT is that it doesn’t need to be in the compilation step. It can be in data files, comments, etc. You can change your syntax highlighting theme to spot the typos x (I added them with a keyboard shortcut anyways)


You need to compile the code with all the debugging logging in it so it can run and print out all the “got here 3”s and so forth.


Because it distinguishes the normal logging function from the ones that are for local debugging.




Consider applying for YC's Spring batch! Applications are open till Feb 11.

Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: