Skip to content
This repository has been archived by the owner on Apr 15, 2019. It is now read-only.

Update copyright and warranty commands to typescript #682

Merged
merged 7 commits into from
Dec 14, 2018

Conversation

shuse2
Copy link
Contributor

@shuse2 shuse2 commented Dec 6, 2018

Description

Updating copyright and warranty commands to TypeScript.

Review checklist

@shuse2
Copy link
Contributor Author

shuse2 commented Dec 10, 2018

esModuleInterop fixed by oclif/config#61

@shuse2 shuse2 mentioned this pull request Dec 13, 2018
12 tasks
Copy link
Contributor

@ishantiw ishantiw left a comment

Choose a reason for hiding this comment

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

Looks good, just missing few return types

status: string,
publicKey: string,
password?: string,
) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

missing return type

const resolveFlags = (
accumulated: ReadonlyArray<string>,
[key, value]: [string, string | boolean | undefined],
) => {
Copy link
Contributor

Choose a reason for hiding this comment

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

here as well

"globby": "8.0.1",
"husky": "0.14.3",
"lint-staged": "5.0.0",
"mocha": "4.0.1",
"nyc": "11.3.0",
"prettier": "1.9.2",
"rxjs-compat": "6.3.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

is this being used?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is for husky, without this it seems like it's not working.
I have no idea why it doesn't work though...

@shuse2 shuse2 changed the base branch from 662-update_transaction_cmd_ts to 649-convert_to_typescript December 14, 2018 11:34
@shuse2 shuse2 merged commit dbfa571 into 649-convert_to_typescript Dec 14, 2018
@shuse2 shuse2 deleted the 680-convert_info_cmd_ts branch December 14, 2018 13:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants