Aram Akhavan

Results 10 comments of Aram Akhavan

@henridb - any luck debugging this? I'm seeing the same

> See https://github.com/sphinx-doc/sphinx-autobuild#relevant-sphinx-bugs. I guess #142 should be reverted/amended since it sounds like rebuilds aren't detecting rescans of source code. > > Can you confirm what version of Sphinx you're...

> > I'm guessing the reporter is right that because the interpreter lives between rebuilds, it's not actually re-importing the updated source code file. > > Hmm... It doesn't --...

Did some more digging; the issue isn't with autobuild. It runs `sphinx-build -v` which doesn't detect the changes in the file. Seems to be fixed by https://github.com/sphinx-doc/sphinx/pull/11939 which has been...

@pradyunsg - there is definitely a `sphinx-autobuild` specific issue here. I installed sphinx from github, so the latest code, and I'm seeing the following: 1. rm -rf docs/_build 2. sphinx-build...

I found another similar error that may be helpful in debugging. If I update `return_type` to be `type[T] | None`, then add a separate check for `result` being none, it...

> This should be solved by using [type adapters](https://docs.pydantic.dev/latest/concepts/type_adapter/). Let me know if this suits your use case. Sort of. But because we have libraries using protocols, it needs to...

I forgot the other reason type adapter didn't work - the protocol I need to satisfy has a class method. Easy enough with `BaseModel` since I can just do `return...

> I provided `TypeAdapter` as a solution as it exactly fits your `ListOf` example, but I'm not sure I understand how you are using protocols. Do you have a concrete...

> Well you did annotate `result` as a `list[Foo]` so that's why I assumed. Fair enough. That was an option we considered in a general sense but discarded for the...