Skip to content
This repository has been archived by the owner on Feb 21, 2024. It is now read-only.

Fix yarn install && yarn build #3

Open
jbowen93 opened this issue Nov 4, 2022 · 0 comments · Fixed by #2
Open

Fix yarn install && yarn build #3

jbowen93 opened this issue Nov 4, 2022 · 0 comments · Fixed by #2

Comments

@jbowen93
Copy link
Member

jbowen93 commented Nov 4, 2022

There is an issue caused by pino-std-serializer and @types/pino-std-serializers where the latter is imported but doesn't contain any actual type definitions. Instead it imports the former into a sub directory.

This results in this error

yarn build
yarn run v1.22.19
$ yarn lerna run build
$ /Users/joshbowen/Code/astriaorg/temp/optimism/node_modules/.bin/lerna run build
lerna notice cli v4.0.0
lerna info versioning independent
lerna info Executing command in 10 packages: "yarn run build"
lerna info run Ran npm script 'build' in '@eth-optimism/core-utils' in 1.4s:
$ tsc -p tsconfig.json
lerna ERR! yarn run build exited 2 in '@eth-optimism/hardhat-deploy-config'
lerna ERR! yarn run build stdout:
$ tsc -p tsconfig.json
error TS2688: Cannot find type definition file for 'pino-std-serializers'.
  The file is in the program because:
    Entry point for implicit type library 'pino-std-serializers'
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
lerna ERR! yarn run build stderr:
error Command failed with exit code 2.
lerna ERR! yarn run build exited 2 in '@eth-optimism/hardhat-deploy-config'
lerna WARN complete Waiting for 1 child process to exit. CTRL-C to exit immediately.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.
error Command failed with exit code 2.
info Visit https://yarnpkg.com/en/docs/cli/run for documentation about this command.

See: microsoft/TypeScript#27956 (comment)

This can be manually resolved by moving pino-std-serializer up a level and deleting @types/pino-std-serializers

This may be related to the below issue/PR:
pinojs/pino-std-serializers#30
pinojs/pino#910

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 a pull request may close this issue.

1 participant