Sitelet https://github.com/fsprojects/Paket/pull/639
Skip to content

Format commands help - #639

Merged
forki merged 28 commits into
fsprojects:masterfrom
theimowski:format_command_help
Feb 23, 2015
Merged

forki merged 28 commits into
fsprojects:masterfrom
theimowski:format_command_help

Conversation

@theimowski

Copy link
Copy Markdown
Member

This is supposed to auto generate command line help usage for each command, as well as auto generate the command line invocation in markdown.

It's still WIP, as some more commands are left to migrate to new approach

Feedback welcome

@forki

forki commented Feb 21, 2015

Copy link
Copy Markdown
Member

/cc @mexx

Comment thread src/Paket/Commands.fs Outdated

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Please fix typo: adhering to the Paket's

@mexx

mexx commented Feb 21, 2015

Copy link
Copy Markdown
Contributor

Good job @theimowski

@forki

forki commented Feb 22, 2015

Copy link
Copy Markdown
Member

the red appyveyor test should be gone after rebase.

Comment thread Paket.sln

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

is this gone?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

yeah, I treat config as a command so it's now generated from the HelpTexts class.
Eventually we'll be able to fallback to separate md file for each command when this PR is merged

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

cool. just tell me when it's ready to merge - really cool stuff

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

thanks, I think it's ready now - I took it even a step further.

Comment thread src/Paket/Commands.fs

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

@mavnn could you please review the messages for this new command?

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good

@theimowski theimowski changed the title [WIP] Format commands help Format commands help Feb 22, 2015
@forki

forki commented Feb 23, 2015

Copy link
Copy Markdown
Member

tumblr_inline_njhdq1yxtk1raprkq

forki added a commit that referenced this pull request Feb 23, 2015
@forki
forki merged commit 4169e9c into fsprojects:master Feb 23, 2015
@theimowski
theimowski deleted the format_command_help branch February 23, 2015 08:57
@forki

forki commented Feb 23, 2015

Copy link
Copy Markdown
Member

Just tested it. Looks great. Thanks so much.

@theimowski

Copy link
Copy Markdown
Member Author

Thanks good to hear that

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.

4 participants