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

SA1642 does not handle structs. #1686

Closed
jcasale opened this issue Oct 24, 2015 · 5 comments
Closed

SA1642 does not handle structs. #1686

jcasale opened this issue Oct 24, 2015 · 5 comments
Labels

Comments

@jcasale
Copy link

jcasale commented Oct 24, 2015

When a struct has ctor, the inspection forces the usage of "class" instead of "struct".

@sharwell
Copy link
Member

This should have been fixed as part of #418 (1.0.0-alpha005). What version are you using?

@jcasale
Copy link
Author

jcasale commented Oct 25, 2015

Hi,
I am using 1.0.0-beta014 from Nuget. Sorry, I forgot to check for closed issues as well, however it seems to still exist for me.

Thanks.

@sharwell
Copy link
Member

Can you provide sample code which reproduces the issue? The tests include multiple checks for the proper use of struct in the summary text (e.g. these tests), so it looks like the problem is in something we hadn't thought of yet.

Thanks!

@jcasale
Copy link
Author

jcasale commented Oct 25, 2015

Sam,
After creating a sample from my project, I realized the problem was on my end.
Sorry for the noise and I really appreciate the help.

Thanks.

@sharwell
Copy link
Member

No problem, glad to hear it's working for you now 😄

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants