Skip to content

Don't use default separator in tagged logger#7084

Merged
josevalim merged 1 commit intorails:masterfrom
LTe:logger_default_separator
Jul 18, 2012
Merged

Don't use default separator in tagged logger#7084
josevalim merged 1 commit intorails:masterfrom
LTe:logger_default_separator

Conversation

@LTe
Copy link
Copy Markdown
Contributor

@LTe LTe commented Jul 18, 2012

When the default separator is set logger will create incorrect output

$, = "_"
@logger.tagged("BCX", "BDX") { @logger.info "Funky time" }
# => [BCX] _[BDX] Funky time

When the default separator is set logger will create incorrect output
josevalim pushed a commit that referenced this pull request Jul 18, 2012
Don't use default separator in tagged logger
@josevalim josevalim merged commit c08f30f into rails:master Jul 18, 2012
tenderlove added a commit that referenced this pull request Jul 18, 2012
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