Kwpolska / merge_args

Merge signatures of two functions with Advanced Hackery.
BSD 3-Clause "New" or "Revised" License
31 stars 6 forks source link

Adding drop_args and drop_kwonlyargs as additional parameters #10

Closed casabre closed 1 year ago

casabre commented 1 year ago

Hi @Kwpolska, we stumbled across your article some years ago and have implemented your function a bit modified in our test application. Now, I have figured out that you are providing an own package for it.

Thus, in order to keep pace with your upstream changes, I have added our adaptations to your decorator. We explicitly have the need to drop some specific arguments from the merge and thus, I have added the two new parameters. I believe that more users could have that use case.

Finally, thanks for your effort and I would love to see my changes merged to main 😄.