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

SA1313 is reported for record declarations incorrectly #3576

Closed
ssg opened this issue Nov 10, 2022 · 4 comments
Closed

SA1313 is reported for record declarations incorrectly #3576

ssg opened this issue Nov 10, 2022 · 4 comments

Comments

@ssg
Copy link
Contributor

ssg commented Nov 10, 2022

StyleCop version is 1.1.118

internal record struct CharMap(char From, char To);

The names "From" and "To" get reported to be violating parameter name standard, but it's the right way to do it with records: https://learn.microsoft.com/en-us/dotnet/csharp/whats-new/tutorials/records#build-temperature-data

@bjornhellander
Copy link
Contributor

1.1.118 is from 2019. Fixed in 1.2.0-beta.261.

@ssg
Copy link
Contributor Author

ssg commented Nov 13, 2022

Again, why's there no stable release since 2019? Certain environments can disallow beta software.

@bjornhellander
Copy link
Contributor

bjornhellander commented Nov 13, 2022

#3571 links to two other issues where @sharwell has explained his view of the situation.

@ssg
Copy link
Contributor Author

ssg commented Nov 13, 2022

This doesn't repro with 1.2.0-beta.435 either. Closing.

@ssg ssg closed this as completed Nov 13, 2022
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

No branches or pull requests

2 participants