Allow Closing to detect dependent resources passed as kwargs too #636

[federinik]

A small addiction to the work of @StummeJ about issue #633 which allows to benefit of his improvement even for dependencies passed as kwargs (such as my case)

[antongmbl]

Hey, any plans to review and merge this one ?

[federinik]

Friendly reminder that this 2-lines PR (rest is just testing) has been open and unmerged for 3 months now

[jazzthief]

Hey, @federinik! I have recently encountered the same issue as you, plus some more (#702 has some details on one of the problems I faced), and started making changes to _locate_dependent_closing_args() before I found this PR. Since we work on the same bit of functionality, do you think we could merge our changes together into a single PR? I think it would be wise in terms of saving the reviewers' time.

[federinik]

Hi @jazzthief, sure thing, feel free to include what I wrote in a common PR/branch or, if you prefer, I could give you access to my fork of the project. Anyway, I'm not really optimistic about receiving soon updates from the reviewers, this PR has been open a while now, unluckily.

Let me know how I can help!

[jazzthief]

@federinik Thank you! I would really like to somehow preserve this PR and give credit to the fact that you first found this problem. So I think I am going to open a PR to combine all the changes and ask you to change your PR's target branch to mine. This way, I can merge your PR and have control over the description of all the changes I am going to make. Does that work for you?

[federinik]

Hi @jazzthief, sound good to me. Let me know when everything is set up and I'll open a PR to your branch. Thank you!

[jazzthief]

@federinik I've created the PR: #711. The branch I work on is GodelTech:702-nested-resource-resolution.

[federinik]

Changes integrated in PR #711, I'm closing this. Thank you all!