Skip to content

Conversation

@mt-roberto
Copy link
Contributor

@mt-roberto mt-roberto commented Dec 5, 2019

  • Increased typescript coverage

  • Catch errors

  • Bump up to version 1.2.4

@mt-roberto mt-roberto added WIP Work in Progress 💀DO NOT MERGE labels Dec 5, 2019
@mt-roberto mt-roberto self-assigned this Dec 5, 2019
mt-micky
mt-micky previously approved these changes Dec 5, 2019
@mt-roberto mt-roberto force-pushed the refactor-improve-typescript-coverage branch 3 times, most recently from 2cc1846 to 7b1aa0d Compare December 5, 2019 07:54
mt-max
mt-max previously approved these changes Dec 5, 2019
@mt-roberto mt-roberto force-pushed the refactor-improve-typescript-coverage branch 2 times, most recently from 9029725 to 81c8188 Compare December 5, 2019 08:28
@mt-roberto mt-roberto requested review from mt-max and mt-micky December 5, 2019 23:41
@mt-roberto mt-roberto removed WIP Work in Progress 💀DO NOT MERGE labels Dec 8, 2019
@mt-roberto mt-roberto force-pushed the refactor-improve-typescript-coverage branch 2 times, most recently from 035960c to 9e27c36 Compare December 9, 2019 02:04
Copy link
Contributor

@mt-micky mt-micky left a comment

Choose a reason for hiding this comment

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

Let's have a chat f2f to make sure we are taking the right path

@mt-roberto mt-roberto force-pushed the refactor-improve-typescript-coverage branch from 9e27c36 to 1c31ae5 Compare December 9, 2019 04:08
@mt-roberto mt-roberto requested a review from mt-micky December 9, 2019 04:11
mt-micky
mt-micky previously approved these changes Dec 9, 2019
@mt-roberto mt-roberto force-pushed the refactor-improve-typescript-coverage branch from 1c31ae5 to 8f05c8a Compare December 9, 2019 05:39
@mt-roberto mt-roberto requested a review from mt-micky December 9, 2019 05:39
@mt-roberto mt-roberto force-pushed the refactor-improve-typescript-coverage branch from 8f05c8a to 836778a Compare December 9, 2019 05:46
@mt-roberto
Copy link
Contributor Author

@mt-micky Can I merge and bump up later?

@mt-micky
Copy link
Contributor

mt-micky commented Dec 9, 2019

Yes, let's merge. Then we will have to bump the version and release on GitHub and NPM

@mt-roberto mt-roberto merged commit 9496a79 into moneytree:master Dec 9, 2019
@mt-roberto mt-roberto deleted the refactor-improve-typescript-coverage branch December 9, 2019 23:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants