You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Python: all: review usage of kwargs and reduce as much as possible #3642
Kwargs should only be used as a escape route, but not for well-known flows of data. In the long term this will protect us from bugs that arise from kwargs being used for mutiple purposes.
Overall guidance:
if there are known extra arguments being passed, reconsider if kwargs is the best way to do that, a named parameter in another function is likely the better approach
kwargs should be part of methods that are part of classes that are often/likely subclassed, so that the kwargs defined in the subclass do not cause issues, in that case, the kwargs should be clearly documented to be there for that purpose and not used for anything else.
Activity
Address PR review comments and restructure instructions
Add kwargs guidelines from issue microsoft#3642 to CODING_STANDARD.md
.NET: Python: Add AGENTS.md files and update coding standards (#3644)
.NET: Python: Add AGENTS.md files and update coding standards (#3644)
Python: clean up kwargs across agents, chat clients, tools, and sessi…
Python: clean up kwargs across agents, chat clients, tools, and sessi…
Python: clean up kwargs across agents, chat clients, tools, and sessi…
Python: clean up kwargs across agents, chat clients, tools, and sessi…
Python: clean up kwargs across agents, chat clients, tools, and sessi…
Python: clean up kwargs across agents, chat clients, tools, and sessi…
Python: clean up kwargs across agents, chat clients, tools, and sessi…
[BREAKING] Python: clean up kwargs across agents, chat clients, tools…