Skip to content

Support conditional output if BubbleWrap is installed#6

Open
markrickert wants to merge 2 commits intomichaeldv:masterfrom
OTGApps:bubblewrap-debug
Open

Support conditional output if BubbleWrap is installed#6
markrickert wants to merge 2 commits intomichaeldv:masterfrom
OTGApps:bubblewrap-debug

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.

2 participants