Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Specify optional parameters by name only #1163

Closed
lars-reimann opened this issue May 10, 2024 · 2 comments · Fixed by #1307
Closed

Specify optional parameters by name only #1163

lars-reimann opened this issue May 10, 2024 · 2 comments · Fixed by #1307
Assignees
Labels
enhancement 💡 New feature or request

Comments

@lars-reimann
Copy link
Member

Is your feature request related to a problem?

We can currently not enforce that some arguments must be passed by name only. This means, we can never change their order.

Desired solution

The Python solution with * is quite confusing. We could instead enforce that all optional parameters must be passed by name.

Possible alternatives (optional)

No response

Screenshots (optional)

No response

Additional Context (optional)

No response

@lars-reimann lars-reimann added the enhancement 💡 New feature or request label May 10, 2024
@lars-reimann lars-reimann added this to DSL May 10, 2024
@github-project-automation github-project-automation bot moved this to Backlog in DSL May 10, 2024
@lars-reimann lars-reimann added the wontfix This will not be worked on label May 17, 2024
@lars-reimann
Copy link
Member Author

Won't fix this, we'll let the user decide whether to pass arguments by name or position. Since the code generator passes all optional arguments by name in the generated Python code, we can still freely swap optional name-only parameters around there.

@lars-reimann lars-reimann closed this as not planned Won't fix, can't repro, duplicate, stale May 17, 2024
@github-project-automation github-project-automation bot moved this from Backlog to ✔️ Done in DSL May 17, 2024
@lars-reimann
Copy link
Member Author

Reopening this, so we can also reorder parameters in the stubs freely, rather than only be able to append them at the end.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement 💡 New feature or request
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant