Results 124 comments of David Mandelberg

In case it helps anyone, here's some code I got working using the GraphQL API. For what it's worth, the new API feels significantly more powerful than the old API...

Edit: sorry for the noise, this was an issue with my code, not with the API. I was caching all "successful" HTTP requests, including ones that return JSON with an...

See the "Edit: ..." text I added to that comment. It was just a transient issue, probably from too many requests to their backend, not too many requests from me....

https://github.com/dseomn/rock-paper-sand/blob/9a26853a0c7a5c68a536b30e69e2cc16c5e4f572/rock_paper_sand/justwatch.py#L233 looks up details by node ID, which is basically just object type + object ID. E.g., object type "SHOW" and object ID "42" is node ID "ts42". I think...

I don't remember if it's possible to pass variables between the two, but when I wanted to do something similar, I decided to separately log when the command began and...

I'm fine with that approach, and nobody else has expressed an opinion, so I made #16790 to implement that. I'll make another PR to add support for `*.bara.sky` afterwards if...

I don't have a strong opinion one way or the other, and I'm happy to make another PR for the initial proposal. P.S. When I made the PR above I...

It looks like the `macro` function is a generator in the `without context` case and a normal function in the other case. Totally guessing since I'm not familiar with the...

And this is the relevant code, right? https://github.com/pallets/jinja/blob/5ef70112a1ff19c05324ff889dd30405b1002044/src/jinja2/compiler.py#L1068-L1089

I just looked at the compiled code when there's anything else in the macro other than the include statement. It looks like that other stuff appends to `t_1`. So the...