Eric Nielsen
Eric Nielsen
I've submitted PR #221 that allows smart case sensitivity matching by default, and adds parameters to force case insensitive or case sensitive matching. Solution is just 1 LOC bigger than...
Oh, I see #205. Maybe you could consider removing the latest tags for the "unreleased" versions.
@rupa, there's still a process substitution at https://github.com/rupa/z/blob/ea5ec7834398ee2244de26953b3d1ef785d3f942/z.sh#L137 that could affect other shells as reported here.
For reference, this is how to reproduce: ```shell $ cd $(mktemp -d) $ touch .zshrc $ ZDOTDIR=${PWD} HOME=${PWD} exec zsh $ autoload -Uz compinit && compinit -C $ eval "$(zoxide...
Thanks for considering this use case. [PEP 589](https://peps.python.org/pep-0589/#supported-and-unsupported-operations) actually says additional keys in the TypedDict should raise an error by type checkers. > For example, `d['x'] = 1` should generate...
Also, since I'm already taking about breaking changes ☺️, I wonder if it would not be cleared to have `additional_properties` take different values like "fail", "include" and" ignore" (or "exclude")...
Cool! I see [JSON schema](https://json-schema.org/understanding-json-schema/reference/object.html#additional-properties) uses the term allow for additional properties that can be included beyond what's declared in the schema, which is something not possible for dataclasses for...
I see how `additional_properties` differs between `serialize` and `deserialize` : | | `serialize` | `deserialize` | | --- | --- | --- | | `additional_properties=False` | Ignore any additional properties...
So `allow_additional_properties ` is meant to be used as described in [Additional properties / pattern properties](https://wyfo.github.io/apischema/0.18/json_schema/#additional-properties-pattern-properties) only for deserialization and `typed_dict_include_extra_keys` is separately meant to be used for both serialization/deserialization...
FYI this was fixed in the ruff implementation of flake8-return, here: https://github.com/charliermarsh/ruff/pull/3047