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.

Sign up for GitHub

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

Jump to bottom

Suggested positional parameter name should be pascal-cased #48310

Closed
jnm2 opened this issue Oct 4, 2020 · 0 comments · Fixed by #48329
Closed

Suggested positional parameter name should be pascal-cased #48310

jnm2 opened this issue Oct 4, 2020 · 0 comments · Fixed by #48329
Labels
4 - In Review A fix for the issue is submitted for review. Area-IDE Concept-Continuous Improvement IDE-CodeStyle Built-in analyzers, fixes, and refactorings New Language Feature - Records Records
Milestone
16.9

Comments

@jnm2
Copy link
Contributor

jnm2 commented Oct 4, 2020

Version Used: 16.8-p3.2

image

Pascal-cased names should be suggested because these become public properties.

(Also see DotNetAnalyzers/StyleCopAnalyzers#3168 (comment))
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
4 - In Review A fix for the issue is submitted for review. Area-IDE Concept-Continuous Improvement IDE-CodeStyle Built-in analyzers, fixes, and refactorings New Language Feature - Records Records
Projects
None yet
Milestone
16.9
Development

Successfully merging a pull request may close this issue.

Treat record positional parameters as properties Youssef1313/roslyn
3 participants
@jnm2 @jinujoseph @jcouv