Codecov Report

Base: 87.00% // Head: 87.00% // No change to project coverage 👍

Coverage data is based on head (77ab8f2) compared to base (cb83eaf).
Patch coverage: 100.00% of modified lines in pull request are covered.

@@           Coverage Diff           @@
##             main    #8049   +/-   ##
=======================================
  Coverage   87.00%   87.00%           
=======================================
  Files         148      148           
  Lines       18486    18486           
  Branches     2520     2520           
=======================================
  Hits        16083    16083           
  Misses       2123     2123           
  Partials      280      280           

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

Hey @auvipy , any updates on this PR?

I need manual verification from you or Omer

I need manual verification from you or Omer

Ok so I've been reading a bit about this change, and about the original discussion, and I'm not sure this will fix the issue.
Adding the **kwargs will only absorb those errors but if the actual kwargs isn't being used as well, then for example where they used --template, it will not be honored as we don't do anything else in this change beside accepting the new arguments.

The original use-case needs to be tested (manually would be fine too) before we can merge this change, in case I am missing something and the kwargs does get used somehow or that everything works if we just "ignore" this exception by absorbing the extra parameters into kwargs.

Do you have a way to validate this change indeed fixes the issue? @auvipy

@thedrow Hey can you give it a quick look please?
You might be able to judge quickly if we can push/block for testing (as it might not be required).

Thanks!