-
Notifications
You must be signed in to change notification settings - Fork 34
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
Increase the number of environments allowed in the -e flag #34
Comments
I really like the idea, thanks Paulo. |
Piotr, Since you self-assigned this one, I am going to add a conversation that I had with Vadim on the issue, so you can chime in if you need: Vadim: btw about -e - max number of environments limit could be easily changed to some big number, but the idea is to allow multiple -e to increase manageability, correct ? Paulo Ney: The problem -- as I recollect -- and I could be wrong is: Vadim: multiple "-e"s adding new values to the list would be much more convenient, correct ? Paulo Ney: Yes, correct ... Vadim: ok thanks |
I haven't done any work related to this issue. Sorry for confusion. I'm short of time in March, so I will welcome any PR on this. I completely agree with leaving "-e" flag as it is and adding a new flag to "append" new environments to the list. |
Vadim, I think a nice plan here would be to change the number of environment allowed to something like a thousand, to allow people to write scripts with any number of environments they want. Then copy the entire framework of the "-e" flag to another "-a" flag, that we would use to "add" environments to the basic list, and modify if appropriately. PN |
The -e flag to specify an environment list only allows 10 environments and gives an error when you try to place one more to the list:
since the command does not allow one to ADD to the existing list, the new list has to be large (the default list has 20 entries now and growing) making it almost impossible to run.
A nice addition would be to add another flag that would allow one to ADD one (or a few) environments to the existing list - to make the command line small and manageable.
The text was updated successfully, but these errors were encountered: