7

Please Please Please remove your shitty ass 'console.log' before commit i don't need your fucking "debug mode" in production

Comments
  • 1
    Add a linter rule that will throw a fatal exception if present.
  • 0
    @SortOfTested i know but the devs keeps removing for "debuging" even in react native apps :(
  • 0
    @jorgeogj
    You can also add an intermediary to scan for lint ignore statements 😉. Cyberwarfare is half the fun of this job.
  • 1
    @SortOfTested Ensure a nuclear warhead is launched every time someone tries to edit the eslint config
  • 0
    It should be catched during PR review. Or by your QA team.

    It shouldn't be possible for this to go in production by error.
  • 0
    What about before production? I leave those in until we are ready for the first customer facing prod release. The rule is that we tag the console with dev or something that starts with dev so we can find it later.

    console.log(‘dev’, foo);
  • 1
    #ifdef prod/dev?
  • 0
    This is devrant, not a pull request.
Add Comment