Skip to content
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

chore: activate no-unused-vars eslint rule #2037

Merged
merged 1 commit into from
Oct 30, 2020
Merged

chore: activate no-unused-vars eslint rule #2037

merged 1 commit into from
Oct 30, 2020

Conversation

fhinkel
Copy link
Contributor

@fhinkel fhinkel commented Oct 30, 2020

Turn on the rule and fix lint problems.

see #2030

@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Oct 30, 2020
@fhinkel fhinkel changed the title chore: active no-unused-vars eslint rule chore: activate no-unused-vars eslint rule Oct 30, 2020
Copy link
Contributor

@ace-n ace-n left a comment

Choose a reason for hiding this comment

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

LGTM for functions

@fhinkel fhinkel merged commit 77b6874 into master Oct 30, 2020
@fhinkel fhinkel deleted the fhinkel-lint branch November 5, 2020 17:38
This was referenced Jan 11, 2022
This was referenced Mar 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants