Skip to content

Conversation

@deivid-rodriguez
Copy link
Contributor

Description:

During some separate work, I was trying to make sure that rake install doesn't print any warnings in CI.

So what I was doing is:

  • Run the command.
  • Redirect the error to a file.
  • Make sure the file is empty.

To my surprise, the file was full of logging messages from the rake package task.

The underlying issue is ruby/fileutils#49.

But I think we can workaround it with this PR.

Tasks:

  • Describe the problem / feature
  • Write tests
  • Write code to solve the problem
  • Get code review from coworkers / friends

I will abide by the code of conduct.

So that I can add a separate test that doesn't set it.
The logging to $stderr is only happening due to a bug in `FileUtils`.
Logging messages are not errors.
@deivid-rodriguez deivid-rodriguez force-pushed the log_messages_to_stdout branch from 2593e05 to 4d1b665 Compare May 18, 2020 16:54
@deivid-rodriguez deivid-rodriguez merged commit 6f1037c into master May 19, 2020
@deivid-rodriguez deivid-rodriguez deleted the log_messages_to_stdout branch May 19, 2020 15:20
@deivid-rodriguez deivid-rodriguez mentioned this pull request May 21, 2020
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants