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

Update config commands to TypeScript - Closes #656 #672

Merged
merged 9 commits into from
Dec 11, 2018

Conversation

shuse2
Copy link
Contributor

@shuse2 shuse2 commented Nov 29, 2018

Description

Update config commands to TypeScript

Review checklist

@shuse2 shuse2 self-assigned this Nov 29, 2018
@shuse2 shuse2 changed the title 656 update config cmd ts Update config commands to TypeScript - Closes #656 Nov 29, 2018
@shuse2 shuse2 force-pushed the 655-update_block_cmd_ts branch from e55773c to 2cf3e08 Compare December 5, 2018 14:05
@shuse2 shuse2 force-pushed the 655-update_block_cmd_ts branch from 2cf3e08 to a871865 Compare December 5, 2018 14:13
@shuse2 shuse2 force-pushed the 656-update_config_cmd_ts branch from 52096a5 to 2607f13 Compare December 5, 2018 14:23
const setArrayURL = (
config: ConfigOptions,
dotNotation: string,
_: string,
Copy link
Contributor

Choose a reason for hiding this comment

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

this argument is needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, this one is not used but the next inputs is used

@shuse2 shuse2 changed the base branch from 655-update_block_cmd_ts to 649-convert_to_typescript December 11, 2018 13:56
@shuse2 shuse2 merged commit 4662173 into 649-convert_to_typescript Dec 11, 2018
@shuse2 shuse2 deleted the 656-update_config_cmd_ts branch December 13, 2018 13:32
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.

2 participants