Skip to content
This repository was archived by the owner on Mar 29, 2022. It is now read-only.

Add logger level to prod environment#440

Draft
fridaland wants to merge 1 commit intoHacktoberfest:masterfrom
fridaland:logger-levels-production-env
Draft

Add logger level to prod environment#440
fridaland wants to merge 1 commit intoHacktoberfest:masterfrom
fridaland:logger-levels-production-env

Conversation

@fridaland
Copy link
Contributor

Description

The rake task to distribute gifted coupons:gifts.rake, is not properly logging out to the terminal. We need to ensure the Rails.logger will run in production as it does in development.

Requirements to merge

  • Rails.logger works in production
  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code in hard-to-understand areas
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

@johndbritton
Copy link
Contributor

The rake task to distribute gifted coupons:gifts.rake, is not properly logging out to the terminal. We need to ensure the Rails.logger will run in production as it does in development.

Rather than adjust the log level in production, you can just use puts in your rake task.

@fridaland fridaland force-pushed the logger-levels-production-env branch from b744cee to 0a34c02 Compare November 19, 2019 15:37
@fridaland fridaland closed this Nov 19, 2019
@fridaland fridaland force-pushed the logger-levels-production-env branch from 0a34c02 to c8cc8b6 Compare November 19, 2019 15:39
@fridaland fridaland reopened this Nov 19, 2019
@fridaland fridaland force-pushed the logger-levels-production-env branch from 453e885 to 068239f Compare November 19, 2019 15:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants