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

Make all fields in .Update optional #98

Open
GGG-KILLER opened this issue Sep 13, 2022 · 1 comment
Open

Make all fields in .Update optional #98

GGG-KILLER opened this issue Sep 13, 2022 · 1 comment
Assignees
Labels
enhancement New feature or request

Comments

@GGG-KILLER
Copy link
Member

Is your feature request related to a problem? Please describe.
Having to provide every single node for .Update is very annoying and could be better as what we have for LuaSyntaxOptions.With

Describe the solution you'd like
The same solution as applied to LuaSyntaxOptions.With.

Describe alternatives you've considered
Providing every single parameter every time or using a combination of multiple .With* methods which would result in a lot of unnecessary allocations.

Additional context
None.

@GGG-KILLER GGG-KILLER added the enhancement New feature or request label Sep 13, 2022
@GGG-KILLER GGG-KILLER self-assigned this Sep 13, 2022
@GGG-KILLER
Copy link
Member Author

This has no milestone because it is not planned for any specific release as I'm still ironing out the following issues:

  • Backwards compatibility;
  • Issue with consumption from languages that aren't C# and VB that may not support default parameters.

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
None yet
Development

No branches or pull requests

1 participant