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

False negative in unneeded_parentheses_in_closure_argument when using `_` #2017

Closed
marcelofabri opened this Issue Jan 24, 2018 · 0 comments

Comments

Projects
None yet
1 participant
@marcelofabri
Collaborator

marcelofabri commented Jan 24, 2018

New Issue Checklist

Bug Report

Complete output when running SwiftLint, including the stack trace and command used
$ echo "call(arg: { ↓(bar, _) in }" | swiftlint lint --no-cache --use-stdin --enable-all-rules
Loading configuration from '.swiftlint.yml'
Done linting! Found 0 violations, 0 serious in 1 file.

Environment

  • SwiftLint version (run swiftlint version to be sure)? 0.24.2
  • Installation method used (Homebrew, CocoaPods, building from source, etc)? Homebrew
  • Paste your configuration file:
opt_in_rules:
  -unneeded_parentheses_in_closure_argument
  • Are you using nested configurations?
    If so, paste their relative paths and respective contents. No
  • Which Xcode version are you using (check xcode-select -p)? 9.2
  • Do you have a sample that shows the issue? Run echo "[string here]" | swiftlint lint --no-cache --use-stdin --enable-all-rules
    to quickly test if your example is really demonstrating the issue. If your example is more
    complex, you can use swiftlint lint --path [file here] --no-cache --enable-all-rules.
// This should trigger a violation:
call(arg: { (bar, _) in })\n

@marcelofabri marcelofabri added the bug label Jan 24, 2018

@marcelofabri marcelofabri changed the title from False positive in unneeded_parentheses_in_closure_argument when using `_` to False negative in unneeded_parentheses_in_closure_argument when using `_` Jan 24, 2018

marcelofabri added a commit to marcelofabri/SwiftLint that referenced this issue Jan 24, 2018

marcelofabri added a commit to marcelofabri/SwiftLint that referenced this issue Jan 24, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment