Skip to content

OpenAI executor doesn't need the deployment property and it fails when you try to execute any YAML file for this type#222

Closed
0GiS0 wants to merge 1 commit intomicrosoft:mainfrom
0GiS0:main
Closed

OpenAI executor doesn't need the deployment property and it fails when you try to execute any YAML file for this type#222
0GiS0 wants to merge 1 commit intomicrosoft:mainfrom
0GiS0:main

Conversation

@0GiS0
Copy link
Copy Markdown

@0GiS0 0GiS0 commented Apr 14, 2025

This pull request includes a small change to the runtime/prompty/prompty/openai/executor.py file. The change removes the deployment attribute from the __init__ method, which is no longer needed.

OpenAI executor doesn't need the deployment property and it fails when you try to execute any YAML file for this type
@sethjuarez
Copy link
Copy Markdown
Member

OK - makes sense.

@sethjuarez
Copy link
Copy Markdown
Member

Thank you for your contribution! We've migrated Prompty to v2 on main — the codebase has been significantly restructured with AgentSchema-based types, scoped @prompty/* npm packages, and new runtimes.

The v1 code is now preserved on the v1 branch for maintenance. If your change is still relevant for v1, please re-open against the v1 branch. For v2 contributions, please open a new PR against main.

Apologies for any inconvenience, and thanks for your patience!

@sethjuarez sethjuarez closed this Mar 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants