Aaron Pham

run python data.py first

For 22.2 GenConverter now is an alias to Converter. The current Converter is renamed to BaseConverter.

Seems like cattrs is relying on import side effect here. Since GenConverter is defined under [`.converter.py`](https://github.com/python-attrs/cattrs/blob/56eda3719e4203792c0557f8539dca24e15258d4/src/cattrs/converters.py#L840), this means when we have relative imports GenConverter should already be included in `locals()`...

This is just a lack of lambda supports for type inference right? a workaround would look something like: ```python _convert_str: t.Callable[[t.Any], str] = lambda d: str(d) cattr.register_unstructure_hook(Path, _convert_str) ```

cc @Tinche whenever you have time. This is just dx improvement, so it would be nice to include in 22.2. Otherwise there is no rush.

I will close this now since I believe you also update annotations on the copy branch.

sg, let me check for any merge config.

I'm I thought that mypy also accept `# type: ignore` syntax 🤔

got it. I think I also configure it to accept the syntax on pyright end. Removing it makes sense then.

cc @coffee-cup on thoughts about this. Implementation wise is relatively light to maintain.