Skip to content
This repository has been archived by the owner on Jul 17, 2023. It is now read-only.

Commit

Permalink
feat(atlas): upgrade pino to v5
Browse files Browse the repository at this point in the history
  • Loading branch information
robertrossmann committed Apr 5, 2019
1 parent 363ebcd commit 60cbd2d
Show file tree
Hide file tree
Showing 9 changed files with 72 additions and 205 deletions.
221 changes: 37 additions & 184 deletions packages/atlas/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions packages/atlas/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,11 +10,11 @@
"@atlas.js/errors": "^0.4.0",
"@atlas.js/hook": "^2.1.0",
"@atlas.js/service": "^1.2.0",
"@types/pino": "^4.7.0",
"@types/pino": "^5.8.0",
"ajv": "^6.5.0",
"ajv-keywords": "^3.2.0",
"lodash": "^4.17.4",
"pino": "^4.7.0"
"pino": "^5.12.1"
},
"engines": {
"node": ">=8.3.0",
Expand Down
3 changes: 2 additions & 1 deletion packages/atlas/test/action.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import {
FrameworkError,
ValidationError,
} from '@atlas.js/errors'
import { symbols } from 'pino'

class DummyAction extends Action {}

Expand Down Expand Up @@ -56,7 +57,7 @@ describe('Atlas::action()', () => {

expect(args).to.have.property('log')
expect(args.log).to.be.an('object')
expect(args.log.chindings).to.match(/"action":"dummy"/u)
expect(args.log[symbols.chindingsSym]).to.match(/"action":"dummy"/u)
})

it('provides the config object on action constructor argument', () => {
Expand Down
3 changes: 2 additions & 1 deletion packages/atlas/test/hook.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import {
FrameworkError,
ValidationError,
} from '@atlas.js/errors'
import { symbols } from 'pino'

class DummyHook extends Hook {
static observes = 'atlas'
Expand Down Expand Up @@ -58,7 +59,7 @@ describe('Atlas::hook()', () => {

expect(args).to.have.property('log')
expect(args.log).to.be.an('object')
expect(args.log.chindings).to.match(/"hook":"dummy"/u)
expect(args.log[symbols.chindingsSym]).to.match(/"hook":"dummy"/u)
})

it('provides config object on hook constructor argument', () => {
Expand Down
3 changes: 2 additions & 1 deletion packages/atlas/test/service.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@ import {
ValidationError,
} from '@atlas.js/errors'
import Service from '@atlas.js/service'
import { symbols } from 'pino'

class DummyService extends Service {}

Expand Down Expand Up @@ -56,7 +57,7 @@ describe('Atlas::service()', () => {

expect(args).to.have.property('log')
expect(args.log).to.be.an('object')
expect(args.log.chindings).to.match(/"service":"dummy"/u)
expect(args.log[symbols.chindingsSym]).to.match(/"service":"dummy"/u)
})

it('provides config object on service constructor argument', () => {
Expand Down
21 changes: 12 additions & 9 deletions packages/component/package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion packages/component/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@
"bugs": "https://github.com/strvcom/atlas.js/issues",
"contributors": [],
"dependencies": {
"@types/pino": "^4.7.0"
"@types/pino": "^5.8.0"
},
"engines": {
"node": ">=8.3.0",
Expand Down
Loading

0 comments on commit 60cbd2d

Please sign in to comment.