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 filenames in Logger.logger_outputs_to? #51125

Merged
merged 2 commits into from
Feb 20, 2024

Conversation

c960657
Copy link
Contributor

@c960657 c960657 commented Feb 19, 2024

Logger.logger_outputs_to? only supports STDOUT and STDERR, not File objects or plain filenames.

@rafaelfranca rafaelfranca merged commit 5628362 into rails:main Feb 20, 2024
4 checks passed
willnet added a commit to willnet/rails that referenced this pull request Feb 27, 2024
….logger_outputs_to?`

Fixed incorrect documentation for `ActiveSupport::Logger.logger_outputs_to?`. The method expects the first argument to be a Logger object and subsequent variadic arguments to be either IO objects or strings representing file paths.

Also corrected the sample code in CHANGELOG.md, which previously only passed a single argument, not reflecting the correct usage.

related PR: rails#51125
jathayde pushed a commit to jathayde/rails that referenced this pull request Feb 28, 2024
….logger_outputs_to?`

Fixed incorrect documentation for `ActiveSupport::Logger.logger_outputs_to?`. The method expects the first argument to be a Logger object and subsequent variadic arguments to be either IO objects or strings representing file paths.

Also corrected the sample code in CHANGELOG.md, which previously only passed a single argument, not reflecting the correct usage.

related PR: rails#51125
Ridhwana pushed a commit to Ridhwana/rails that referenced this pull request Mar 7, 2024
….logger_outputs_to?`

Fixed incorrect documentation for `ActiveSupport::Logger.logger_outputs_to?`. The method expects the first argument to be a Logger object and subsequent variadic arguments to be either IO objects or strings representing file paths.

Also corrected the sample code in CHANGELOG.md, which previously only passed a single argument, not reflecting the correct usage.

related PR: rails#51125
viralpraxis pushed a commit to viralpraxis/rails that referenced this pull request Mar 24, 2024
….logger_outputs_to?`

Fixed incorrect documentation for `ActiveSupport::Logger.logger_outputs_to?`. The method expects the first argument to be a Logger object and subsequent variadic arguments to be either IO objects or strings representing file paths.

Also corrected the sample code in CHANGELOG.md, which previously only passed a single argument, not reflecting the correct usage.

related PR: rails#51125
fractaledmind pushed a commit to fractaledmind/rails that referenced this pull request May 13, 2024
….logger_outputs_to?`

Fixed incorrect documentation for `ActiveSupport::Logger.logger_outputs_to?`. The method expects the first argument to be a Logger object and subsequent variadic arguments to be either IO objects or strings representing file paths.

Also corrected the sample code in CHANGELOG.md, which previously only passed a single argument, not reflecting the correct usage.

related PR: rails#51125
xjunior pushed a commit to xjunior/rails that referenced this pull request Jun 9, 2024
….logger_outputs_to?`

Fixed incorrect documentation for `ActiveSupport::Logger.logger_outputs_to?`. The method expects the first argument to be a Logger object and subsequent variadic arguments to be either IO objects or strings representing file paths.

Also corrected the sample code in CHANGELOG.md, which previously only passed a single argument, not reflecting the correct usage.

related PR: rails#51125
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants