Skip to content

Commit

Permalink
config: remove LOGGER_ENABLE option
Browse files Browse the repository at this point in the history
LOGGER_LEVEL has 'silent' which disable a log.
Having LOGGER_ENABLE is redundant.

PR-URL: #46
  • Loading branch information
makemek committed Apr 29, 2017
1 parent c56d994 commit cb52dd4
Show file tree
Hide file tree
Showing 3 changed files with 0 additions and 3 deletions.
1 change: 0 additions & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -72,7 +72,6 @@ PORT_DEV=<port number> # browser-sync listeing port; default is 3000
At `env` section in `process.yml` are environment variables that will override `.env.json` for running in production environment.

### Environment Variables
- LOGGER_ENABLE: if `true` enables log, `false` disables log
- LOGGER_LEVEL: select a [logging level](https://github.com/pinojs/pino/blob/master/docs/API.md#level)
- LOGGER_PRETTY: pretty print log (should be set to `false` in production)
- PORT: application port
Expand Down
1 change: 0 additions & 1 deletion process.yml
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ apps:
pid_file: ./log/modcolle.pid

env:
LOGGER_ENABLE: true
LOGGER_PRETTY: true
LOGGER_LEVEL: info
PORT: 5000
Expand Down
1 change: 0 additions & 1 deletion src/logger.js
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,6 @@ const loggers = {}
labels.forEach(label => {
loggers[label] = pino({
name: label,
enabled: LOGGER_ENABLE === 'true',
level: LOGGER_LEVEL,
prettyPrint: LOGGER_PRETTY === 'true'
})
Expand Down

0 comments on commit cb52dd4

Please sign in to comment.