Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add: Commander integration #12

Merged
merged 13 commits into from
Apr 20, 2020
Merged

Add: Commander integration #12

merged 13 commits into from
Apr 20, 2020

Conversation

oliverjiu
Copy link

Description

I added integration of commander.

Issues

@oliverjiu oliverjiu added the enhancement New feature or request label Apr 20, 2020
@oliverjiu oliverjiu added this to the v1.0 milestone Apr 20, 2020
@oliverjiu oliverjiu requested a review from yanocv April 20, 2020 09:36
@oliverjiu oliverjiu added this to In progress in dev via automation Apr 20, 2020
@oliverjiu oliverjiu self-assigned this Apr 20, 2020
This was linked to issues Apr 20, 2020
Copy link
Member

Codacy Here is an overview of what got changed by this pull request:

Issues
======
+ Solved 4
- Added 3
           

See the complete overview on Codacy

});

// Array of text strings to process
let textArray: Promise<Array<string>> = new Promise(function (resolve) {
Copy link
Member

Choose a reason for hiding this comment

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

Codacy Issue found: Delete ·

program.description(`Write arguments to the standard output.

Display the ARGs on the standard output followed by a newline.`);
program.on("--help", function () {
Copy link
Member

Choose a reason for hiding this comment

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

Codacy Issue found: Delete ·

// Array of text strings to process
let textArray: Promise<Array<string>> = new Promise(function (resolve) {
// Get strings; The method action is synchronous.
program.arguments("[strings...]").action(function (argStrings) {
Copy link
Member

Choose a reason for hiding this comment

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

Codacy Issue found: Delete ·

Copy link
Contributor

@yanocv yanocv left a comment

Choose a reason for hiding this comment

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

All good.

@yanocv yanocv merged commit def607e into dev Apr 20, 2020
dev automation moved this from In progress to Done Apr 20, 2020
@yanocv yanocv deleted the dev-commander branch April 20, 2020 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
dev
  
Done
Development

Successfully merging this pull request may close these issues.

Add: "-n" flag Add: Commander initialization
4 participants