Skip to content

Commit

Permalink
build: fix typings and webpack errors (#825)
Browse files Browse the repository at this point in the history
  • Loading branch information
AuHau authored Jan 31, 2023
1 parent d9137b9 commit 24ca84f
Show file tree
Hide file tree
Showing 34 changed files with 167 additions and 85 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/check.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -53,6 +53,9 @@ jobs:
- name: Check typings
run: npm run check:types

- name: Check build
run: npm run build

- name: Trigger API Docs update PR
uses: peter-evans/repository-dispatch@v1
if: github.ref == 'refs/heads/master'
Expand Down
171 changes: 98 additions & 73 deletions package-lock.json

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

6 changes: 3 additions & 3 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -122,10 +122,10 @@
"sinon-chai": "^3.7.0",
"terser-webpack-plugin": "^5.3.1",
"ts-node": "^10.9.1",
"typescript": "^4.9.3",
"webpack": "^5.72.1",
"typescript": "^4.9.5",
"webpack": "^5.75.0",
"webpack-bundle-analyzer": "^4.5.0",
"webpack-cli": "^4.9.2"
"webpack-cli": "^5.0.1"
},
"engines": {
"node": ">=14.0.0",
Expand Down
3 changes: 3 additions & 0 deletions src/bee-debug.ts
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,10 @@ import {
} from './types'
import * as tag from './modules/debug/tag'
import * as stamps from './modules/debug/stamps'

// @ts-ignore: Needed TS otherwise complains about importing ESM package in CJS even though they are just typings
import type { Options as KyOptions } from 'ky-universal'

import { DEFAULT_KY_CONFIG, wrapRequestClosure, wrapResponseClosure } from './utils/http'
import { sleep } from './utils/sleep'
import { deepMerge } from './utils/merge'
Expand Down
1 change: 1 addition & 0 deletions src/bee.ts
Original file line number Diff line number Diff line change
Expand Up @@ -78,6 +78,7 @@ import {
UploadResult,
} from './types'

// @ts-ignore: Needed TS otherwise complains about importing ESM package in CJS even though they are just typings
import type { Options as KyOptions } from 'ky'
import { DEFAULT_KY_CONFIG, wrapRequestClosure, wrapResponseClosure } from './utils/http'
import { isReadable } from './utils/stream'
Expand Down
2 changes: 2 additions & 0 deletions src/chunk/soc.ts
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,8 @@ import * as socAPI from '../modules/soc'
import * as chunkAPI from '../modules/chunk'
import { EthAddress } from '../utils/eth'
import { assertAddress } from '../utils/type'

// @ts-ignore: Needed TS otherwise complains about importing ESM package in CJS even though they are just typings
import type { Options as KyOptions } from 'ky'

const IDENTIFIER_SIZE = 32
Expand Down
2 changes: 2 additions & 0 deletions src/feed/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -21,6 +21,8 @@ import { bytesToHex, hexToBytes, HexString, makeHexString } from '../utils/hex'
import { readUint64BigEndian, writeUint64BigEndian } from '../utils/uint64'
import * as chunkAPI from '../modules/chunk'
import { EthAddress, HexEthAddress, makeHexEthAddress } from '../utils/eth'

// @ts-ignore: Needed TS otherwise complains about importing ESM package in CJS even though they are just typings
import type { Options as KyOptions } from 'ky'

import type { FeedType } from './type'
Expand Down
Loading

0 comments on commit 24ca84f

Please sign in to comment.