JoinColony / purser

Interact with Ethereum wallets easily
https://joincolony.github.io/purser
MIT License
84 stars 21 forks source link

Move this project to TypeScript #287

Open chmanie opened 4 years ago

chmanie commented 4 years ago

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:

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.

SurfingNerd commented 4 years ago

Hello, i am interested in doing this :), i could start in about 1-2 weeks. (around february 25th). could you clarify if a trezor and ledger is needed for doing the tests ?

chmanie commented 4 years ago

Hey @SurfingNerd! Sure, feel free to start on this any time! I'll assign you to this issue here in github. Can you request to work on it in Colony is well, so you'll get the bounty once you're done? What's your Colony username?

A Trezor or a Ledger is not needed to run the current test suite.

befitsandpiper commented 4 years ago

Hi, is this task still open? I'm seeing messages of people saying it's done on BetaColony, and it's assigned to SurfingNerd here, but also all the updates seem super old.

chmanie commented 4 years ago

@befitsandpiper we are still missing the ledger and trezor plugins in the typescript branch. You can follow the progress here: https://github.com/JoinColony/purser/pull/322 If you feel like you want to contribute to this task, feel free to get in touch!