search

argos-ci / argos-cli

Argos command line interface.
https://www.argos-ci.com
MIT License
12 stars 4 forks source link

feat(UI): better messages #9

Closed oliviertassinari closed 7 years ago

oliviertassinari commented 7 years ago

Closes https://github.com/argos-ci/argos/issues/262

capture d ecran 2017-04-24 a 01 20 33

gregberge commented 7 years ago

You can't put the notion of environment in a CLI tool, it doesn't make any sense.

oliviertassinari commented 7 years ago

You can't put the notion of environment in a CLI tool, it doesn't make any sense.

It does make sense for me. We might want different behavior:

How would you handle that without the notion of environment? Still, I agree that users should always be using the production environment, that's exactly the same for a web app.

oliviertassinari commented 7 years ago

Fixed.

gregberge commented 7 years ago

You should use an environment variable to target a different endpoint but not the environment itself. On Mon, 24 Apr 2017 at 16:32, Olivier Tassinari notifications@github.com wrote:

Merged #9 https://github.com/argos-ci/argos-cli/pull/9.

— You are receiving this because you were assigned. Reply to this email directly, view it on GitHub https://github.com/argos-ci/argos-cli/pull/9#event-1054882922, or mute the thread https://github.com/notifications/unsubscribe-auth/AAQQPnoHBHa6MLMhbeFaqPi-6f06MBsDks5rzLKZgaJpZM4NFkXl .