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

no-console: fix #3207 #4041

Merged
merged 1 commit into from
Jul 16, 2018
Merged

no-console: fix #3207 #4041

merged 1 commit into from
Jul 16, 2018

Conversation

shalomdotnet
Copy link
Contributor

PR checklist

Overview of change:

In the code below:

let x = [1, 2, 3];
console.log(x);
x.forEach(console.log);

The linter not warns only on line 2, but also on line 3

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

CHANGELOG.md entry:

@palantirtech
Copy link
Member

Thanks for your interest in palantir/tslint, @shalomdotnet! 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.

Copy link
Contributor

@johnwiseheart johnwiseheart left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for contributing!

@johnwiseheart johnwiseheart merged commit 1eb6640 into palantir:master Jul 16, 2018
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.

4 participants