Skip to content
This repository was archived by the owner on Sep 30, 2021. It is now read-only.

fix(types)!: polkadot/api to v4 w. esm; Use substrate/dev for build#404

Merged
emostov merged 3 commits intomasterfrom
zeke-use-substrate-dev
Mar 12, 2021
Merged

fix(types)!: polkadot/api to v4 w. esm; Use substrate/dev for build#404
emostov merged 3 commits intomasterfrom
zeke-use-substrate-dev

Conversation

@emostov
Copy link
Contributor

@emostov emostov commented Mar 10, 2021

BREAKING CHANGE

Emitted .js is now es2019, requiring node >= 12.

Changes

  • Use @substrate/dev for build configuration
    • new tsconfig.json targets es2019
    • prettier now uses hard tabs and 2 spaces instead of soft tabs and 2 spaces, which causes the huge diff
    • some slight type annotations where necessary for claims.ts and method.ts in order to pass the stricter tsconfig
    • Babel is now used to transpile the @polkadot/* packages .js from ESM to CJS for jest

## BREAKING CHANGE
Emitted .js is now es2020, requiring node >= 12.

## Changes
- Use @substrate/dev for build configuration
    - new tsconfig.json targets es2019
    - prettier is now uses hard tabs and 2 spaces, which causes the huge diff
    - some slight type annotations where necessary for claims.ts and method.ts in order to pass the stricter tsconfig
    - Babel is now used to transpile the @polkadot/* packages `.js` from ESM to CJS for jest
Copy link
Member

@TarikGul TarikGul left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. One nit pick

Co-authored-by: Tarik Gul <47201679+TarikGul@users.noreply.github.com>
@emostov emostov requested review from dvdplm and removed request for dvdplm March 11, 2021 18:03
@emostov emostov merged commit 2280218 into master Mar 12, 2021
@emostov emostov deleted the zeke-use-substrate-dev branch March 12, 2021 14:15
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 this pull request may close these issues.

3 participants