Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support conditional output if BubbleWrap is installed #6

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Support conditional output if BubbleWrap is installed #6

wants to merge 2 commits into from

Conversation

markrickert
Copy link

This is a backwards compatible and non-breaking change that does the following:

  1. Detects if the user is using the BubbleWrap gem.
  2. If not, it does the same thing it has always done.
  3. If so, it checks to see if BW.debug is set to true.
  4. If not, the the ap call will not output anything.
  5. If so, it will allow the console output to proceed as normal.

This is based on my blog post: http://blog.markrickert.me/leaving-debug-code-in-production-apps and I figured that making it automatic would be useful.

I hope you merge this and release a new gem version! I know a few people that have been pestering me to implement this functionality in this gem.

cc/ @GantMan

@GantMan
Copy link

GantMan commented May 15, 2014

:shipit:

@markrickert markrickert changed the title Suppor conditional output if BubbleWrap is installed Support conditional output if BubbleWrap is installed May 23, 2014
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.

None yet

2 participants