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

Move this project to TypeScript #287

@chmanie

Description

@chmanie

Payout

Heads up! There's a bounty on this task! Go to https://colony.io/colony/beta/task/5e4566a3cbb233006996c3a7 learn more!

Description

Issue owner: chris@colony.io

To be in line with our other projects in Colony, this library should be move to TypeScript.

These should be the minimal configs setting the "strictness" of the TS compiler (compilerOptions). Relaxation of these rules could be allowed if necessary (please consult the issue owner before making those changes).

    "lib": ["es2018"],
    "module": "commonjs",
    "moduleResolution": "node",
    "strictNullChecks": true,
    "strictPropertyInitialization": true,
    "target": "es6",

This issue is regarded done when:

  • All the code of all modules (purser-core, purser-software, purser-metamask, purser-ledger, purser-trezor) was converted to TypeScript and the test suite passes
  • The tooling was adjusted so that we can still easily build the project
  • The library was manually tested using purser-software (and a video is provided).

Bonus points: (the infamous 🌮- bonus):

Add the noExplicitAny compiler option and adhere to it. This might involve stubbing out types for the external libraries that we're using.

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions