Skip to content

ESLint: Enable computed property rules #4077

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
Merged
merged 1 commit into from
Oct 22, 2021
Merged

ESLint: Enable computed property rules #4077

merged 1 commit into from
Oct 22, 2021

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Oct 22, 2021

With #4068 we can finally enable these rules now 🥳

@Turbo87 Turbo87 added A-frontend 🐹 C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear labels Oct 22, 2021
@Turbo87 Turbo87 requested review from locks and pichfl October 22, 2021 13:11
@locks
Copy link
Contributor

locks commented Oct 22, 2021

@bors r+

@bors
Copy link
Contributor

bors commented Oct 22, 2021

📌 Commit 75eff3e has been approved by locks

@bors
Copy link
Contributor

bors commented Oct 22, 2021

⌛ Testing commit 75eff3e with merge 20d218c...

@bors
Copy link
Contributor

bors commented Oct 22, 2021

☀️ Test successful - checks-actions
Approved by: locks
Pushing 20d218c to master...

@bors bors merged commit 20d218c into rust-lang:master Oct 22, 2021
@Turbo87 Turbo87 deleted the eslint-computed branch October 24, 2021 19:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@locks locks Awaiting requested review from locks

@pichfl pichfl Awaiting requested review from pichfl

Assignees
No one assigned
Labels
A-frontend 🐹 C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@Turbo87 @locks @bors