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

SA1008 is shown for range expressions #3040

Closed
ssg opened this issue Oct 15, 2019 · 0 comments · Fixed by #3041
Closed

SA1008 is shown for range expressions #3040

ssg opened this issue Oct 15, 2019 · 0 comments · Fixed by #3041
Assignees
@ssg
Labels
bug c# 8 fixed
Milestone
1.2.0-beta.113

Comments

@ssg
Copy link
Contributor

ssg commented Oct 15, 2019 •
edited

The code below produces SA1008 "Opening parenthesis should be preceded by space" which I think shouldn't happen because double dots already give enough leeway for readability:

return output[..(finalOutputLen - 1)];

With a preceding space, it just looks like a misformatted source code.

StyleCop Version: 1.1.118, .netstandard2.1, .NET Core 3.0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ssg ssg

Labels
bug c# 8 fixed
Projects
None yet
Milestone
1.2.0-beta.113
Development

Successfully merging a pull request may close this issue.

Do not require space before open parenthesis in a range expression (#3040)
2 participants
@ssg @sharwell