Skip to content

Conversation

@toddpress
Copy link

  1. create new helper that returns a conditional console.log
  2. rename that helper
  3. refactor blocks like this: if (this.DEBUG) console.log('foo') -> logInDebug('foo')
  4. Will change logger alias to logIf instead of logIn, which could be confusing in an authenticated context

@wyattbenno777
Copy link

Hey,
Thanks. This all looks ok other than the "lol" portion :)
Is it working fine on local for you?

@toddpress
Copy link
Author

other than the jsDoc still being wrong. 😫

@toddpress
Copy link
Author

@wyattbenno777 yep, looking good! Found some serious utility in this lib today, wanted to give back.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants