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

Friendlier warnings about obsolete syntax #2610

Merged
merged 1 commit into from Aug 15, 2017

Conversation

Projects
None yet
3 participants
@TeaDrivenDev
Contributor

TeaDrivenDev commented Aug 13, 2017

When using the "old" syntax for Paket commands, such as paket add nuget <package>, Paket shows a warning saying

Please use the new syntax: nuget -> (omit, option is the new default argument)

or

Please use the new syntax: project -> --project

These are concise but not very friendly, and the first one requires some interpretation to make sense of.

This PR makes these messages more human-readable:

Please use the new syntax: 'nuget' is the default argument and should be omitted.

and

Please use the new syntax: 'project' has been replaced by '--project'.

Also, the error message when using old and new syntax together has been changed from

You cannot use the old and new syntax at the same time: project <-> --project

to

You cannot use the old and new syntax at the same time: 'project' has been replaced by '--project'. You cannot mix the two.

@forki forki requested a review from agross Aug 15, 2017

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Aug 15, 2017

Member

thx!

Member

forki commented Aug 15, 2017

thx!

@forki forki merged commit 31e074c into fsprojects:master Aug 15, 2017

0 of 2 checks passed

continuous-integration/travis-ci/pr The Travis CI build could not complete due to an error
Details
continuous-integration/appveyor/pr AppVeyor build failed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment