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

Move UnsetValue to a consts (or similar) module #53

Open
mih opened this issue Oct 17, 2024 · 0 comments
Open

Move UnsetValue to a consts (or similar) module #53

mih opened this issue Oct 17, 2024 · 0 comments

Comments

@mih
Copy link
Member

mih commented Oct 17, 2024

This is a useful and generic type, that can be reused in more contexts than just settings.

And when this is touched, actually derive UnsetValue from inspect.Parameter.empty. This is the place in the standard lib that provides this concept.

mih added a commit to datalad/datalad-core that referenced this issue Oct 22, 2024
It makes sense to promote this special type in a central place to avoid
proliferation of equivalent alternatives.

Also see: datalad/datasalad#53

Also add an `__all__`.
mih added a commit to datalad/datalad-core that referenced this issue Oct 22, 2024
It makes sense to promote this special type in a central place to avoid
proliferation of equivalent alternatives.

Also see: datalad/datasalad#53

Also add an `__all__`.
mih added a commit to datalad/datalad-core that referenced this issue Oct 24, 2024
It makes sense to promote this special type in a central place to avoid
proliferation of equivalent alternatives.

Also see: datalad/datasalad#53

Also add an `__all__`.
mih added a commit to datalad/datalad-core that referenced this issue Oct 25, 2024
It makes sense to promote this special type in a central place to avoid
proliferation of equivalent alternatives.

Also see: datalad/datasalad#53

Also add an `__all__`.
mih added a commit to datalad/datalad-core that referenced this issue Oct 25, 2024
It makes sense to promote this special type in a central place to avoid
proliferation of equivalent alternatives.

Also see: datalad/datasalad#53

Also add an `__all__`.
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

1 participant