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

Remove naming constraints in auto-client. #4237

Merged
merged 1 commit into from
Aug 2, 2023
Merged

Conversation

geeknoid
Copy link
Member

@geeknoid geeknoid commented Aug 2, 2023

  • The auto-client generator no longer disallows method names and parameter names that start with _. This was previously done to avoid naming conflicts with identifiers created by the generator. Now, the generator dynamically picks new names if there are conflicts with user-defined symbols.
Microsoft Reviewers: Open in CodeFlow

@ghost ghost assigned geeknoid Aug 2, 2023
Copy link
Contributor

@dbarbosapn dbarbosapn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, just curious about one thing. Left a comment :)

@ghost ghost added the waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue. label Aug 2, 2023
- The auto-client generator no longer disallows method names
and parameter names that start with _. This was previously done to
avoid naming conflicts with identifiers created by the generator. Now,
the generator dynamically picks new names if there are conflicts with
user-defined symbols.
@geeknoid geeknoid force-pushed the geeknoid/autoclient branch from 8f82aaf to 94f1183 Compare August 2, 2023 17:48
@ghost ghost removed the waiting-author-feedback 📭 The author of this issue needs to respond in order for us to continue investigating this issue. label Aug 2, 2023
Copy link
Contributor

@dbarbosapn dbarbosapn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@geeknoid geeknoid merged commit 44357be into main Aug 2, 2023
@geeknoid geeknoid deleted the geeknoid/autoclient branch August 2, 2023 21:14
@ghost ghost added this to the 8.0 RC1 milestone Aug 2, 2023
@geeknoid geeknoid mentioned this pull request Aug 2, 2023
@ghost ghost locked as resolved and limited conversation to collaborators Sep 2, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants