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

Set/Add on list params #1150

Open
aolofsson opened this issue Sep 29, 2022 · 1 comment
Open

Set/Add on list params #1150

aolofsson opened this issue Sep 29, 2022 · 1 comment
Assignees

Comments

@aolofsson
Copy link
Member

I have made this mistake a few times now, (with option, ydir), and each time it's a few minutes gone. I know we have gone back and forth on this, but I would think that doing clearing out a list by using set list param that has already been set, should be a rare occurrence for the user? It would be great to get some feedback along the lines of "hey, did you really mean to do this?".

@aolofsson aolofsson changed the title Set/Add on values Set/Add on list params Sep 29, 2022
@nmoroze
Copy link
Contributor

nmoroze commented Sep 29, 2022

Makes sense, we could make this a warning (maybe with a flag to quiet it as well). I think we'll need to fix #1146 first for this to make sense though, otherwise we'll get a warning on everything that has a default value.

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

No branches or pull requests

2 participants