Web3.js: Add TypeScript typings for all packages

Created on 30 Nov 2018  ·  30Comments  ·  Source: ChainSafe/web3.js

This is a gitcoin bounty.

ToDo:

  • [x] Add type definitions for all modules
  • [x] Add dtslint tests for all modules
  • [x] Add typings to PR template check list
  • [x] See if there's a coverage tool for dtslint tests
bounty

All 30 comments

Already on this as we know (as I was the one who suggested this change). Will wait for bounty to be created but will carry on with the tasks 😃

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__This issue now has a funding of 600.0 DAI (600.0 USD @ $1.0/DAI) attached to it as part of the Ethereum Foundation fund.__

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__Work has been started__.

These users each claimed they can complete the work by 2 months ago.
Please review their action plans below:

1) joshstevens19 has been approved to start work.

Hey i was the one who initially thought of this issue and suggested it to Sam.

I have already done 2 PR for 2 packages and think i would be the correct suited for this job!

Learn more on the Gitcoin Issue Details page.

2) huanzhang89 has applied to start work _(Funders only: approve worker | reject worker)_.

Will be helping Josh on this. First time using gitcoin, just trying this out :)

Learn more on the Gitcoin Issue Details page.

3) samthomson has applied to start work _(Funders only: approve worker | reject worker)_.

I have experience working with TypeScript including projects which moved from JS to Typescript. Would be happy to help out with this.

Learn more on the Gitcoin Issue Details page.

Just approved you @joshstevens19 glad to have you helping out :)

Cheers Scott 👍

@joshstevens19 Hello from Gitcoin Core - are you still working on this issue? Please submit a WIP PR or comment back within the next 3 days or you will be removed from this ticket and it will be returned to an ‘Open’ status. Please let us know if you have questions!

  • [x] reminder (3 days)
  • [ ] escalation to mods (6 days)

Funders only: Snooze warnings for 1 day | 3 days | 5 days | 10 days | 100 days

Yes I have already done 3 PRs which have been merged for 3 packages, will be doing the other ones when I get a second! 😁

@ceresstation This bounty is now done :)
Thanks to @joshstevens19 for this awesome work!

@ceresstation I have submitted the work on gitcoin 👍

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__Work for 600.0 DAI (600.0 USD @ $1.0/DAI) has been submitted by__:

  1. @joshstevens19

@ceresstation please take a look at the submitted work:

  • PR by @joshstevens19

@ceresstation hey when will the gitcoin bounty be paid out ?

Hey @joshstevens19 sorry for the delay here, paying you out now :)

Javascript Master Scriber ⚡️ A *Javascript Master Scriber* Kudos has been sent to @joshstevens19 for this issue from @ceresstation. ⚡️ Nice work @joshstevens19! Your Kudos has automatically been sent in the ETH address we have on file.

Issue Status: 1. Open 2. Started 3. Submitted 4. Done


__The funding of 600.0 DAI (600.0 USD @ $1.0/DAI) attached to this issue has been approved & issued to @joshstevens19.__

Was this page helpful?
0 / 5 - 0 ratings