Skip to content

ESLint: Enable disabled rules #2043

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

Merged
merged 1 commit into from
Dec 21, 2019
Merged

ESLint: Enable disabled rules #2043

merged 1 commit into from
Dec 21, 2019

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Dec 21, 2019

... and instead only disable specific cases using eslint-disable-next-line comments

Resolves #2042

r? @jtgeibel

... and instead only disable specific cases using `eslint-disable-next-line` comments
@jtgeibel
Copy link
Member

LGTM, thanks!

@bors r+

@bors
Copy link
Contributor

bors commented Dec 21, 2019

📌 Commit b51cf94 has been approved by jtgeibel

bors added a commit that referenced this pull request Dec 21, 2019
ESLint: Enable disabled rules

... and instead only disable specific cases using `eslint-disable-next-line` comments

Resolves #2042

r? @jtgeibel
@bors
Copy link
Contributor

bors commented Dec 21, 2019

⌛ Testing commit b51cf94 with merge e06f976...

@bors
Copy link
Contributor

bors commented Dec 21, 2019

☀️ Test successful - checks-travis
Approved by: jtgeibel
Pushing e06f976 to master...

@bors bors merged commit b51cf94 into rust-lang:master Dec 21, 2019
@Turbo87 Turbo87 deleted the eslint branch December 21, 2019 23:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Actions displays frontend lints to every PR
4 participants