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

make DTConfig attributes lists, not sets #49

Open
ev-br opened this issue Jul 9, 2022 · 0 comments
Open

make DTConfig attributes lists, not sets #49

ev-br opened this issue Jul 9, 2022 · 0 comments

Comments

@ev-br
Copy link
Member

ev-br commented Jul 9, 2022

Convert to sets only when used. The reason is config.rndm_markers += ["# uninitialized"] is reasonably expected to work but fails because it's a set, hence the it needs to be spelled as .add("# uninitialized") etc. And that's clunky and is likely to trip users.

@ev-br ev-br added maintenance enhancement New features w.r.t. the original refguide-check and removed maintenance labels Jul 9, 2022
@ev-br ev-br added maintenance and removed enhancement New features w.r.t. the original refguide-check labels Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant