Skip to content
This repository has been archived by the owner on Mar 25, 2021. It is now read-only.

Clarified lint message "All imports are unused." #3831

Merged
merged 1 commit into from
Apr 16, 2018

Conversation

jishi9
Copy link
Contributor

@jishi9 jishi9 commented Apr 12, 2018

Changed to: "All imports on this line are unused"

PR checklist

  • Addresses an existing issue: #0000
  • New feature, bugfix, or enhancement
    • Includes tests
  • Documentation update

Overview of change:

Clarified lint message for "All imports are unused." , which suggests that all imports in the file are unused, when the intention is all imports in the line.

Is there anything you'd like reviewers to focus on?

CHANGELOG.md entry:

Changed to: "All imports on this line are unused"
@palantirtech
Copy link
Member

Thanks for your interest in palantir/tslint, @jishi9! Before we can accept your pull request, you need to sign our contributor license agreement - just visit https://cla.palantir.com/ and follow the instructions. Once you sign, I'll automatically update this pull request.

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

Successfully merging this pull request may close these issues.

3 participants