Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Commit d2aa568

Browse files
committed
Merge branch 'change/remove-some-lodash' into 'master'
Remove some lodash packages See merge request Backend/ackee-node-logger!9
2 parents 8a381b7 + 994b72b commit d2aa568

6 files changed

Lines changed: 1565 additions & 1584 deletions

File tree

CHANGELOG.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,12 @@
1+
## [0.2.6] - 2018-10-04
2+
13
### Added
24
- log version of `ackee-node-logger` package being used to `pkgVersion` field
35

46
### Changed
57
- change default serializer for `process` object
68
- do not log undefined and empty objects in default serializers
9+
- remove some lodash packages
710

811
## [0.2.5] - 2018-09-10
912

index.js

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
const get = require('lodash.get');
2-
const merge = require('lodash.merge');
31
const isString = require('lodash.isstring');
42
const isObject = require('lodash.isobject');
53
const pino = require('pino');
@@ -73,7 +71,7 @@ const defaultLogger = (options = {}) => {
7371
{ level: levels.warn, stream: process.stderr },
7472
];
7573
}
76-
if (!get(options, 'disableStackdriverFormat', false)) {
74+
if (!options.disableStackdriverFormat) {
7775
streams = decorateStreams(streams, StackDriverFormatStream);
7876
}
7977

@@ -84,7 +82,9 @@ const defaultLogger = (options = {}) => {
8482
}
8583

8684
const logger = pino(
87-
merge(
85+
// no deep-merging needed, so assign is OK
86+
Object.assign(
87+
{},
8888
{
8989
level: defaultLevel,
9090
timestamp: false,

0 commit comments

Comments
 (0)