-
-
Notifications
You must be signed in to change notification settings - Fork 3.5k
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
lint(): fix eslint errors #8729
Conversation
@ShaMan123 Sorry, for wasting your time earlier, I closed the previous PR and now I open a new PR to merge into the master branch. Sorry for always wasting your time with simple questions. |
You are being too harsh on yourself. |
take a look at the failing tests and fix what is wrong. |
Then once all tests pass (lint can fail) request my review |
@ShaMan123 ok, I will try, but My time zone +8, I'll try to fix this tomorrow |
@ShaMan123 Can you still execute CI? I seem to run |
seems you didn't install deps |
dump
|
@ShaMan123 I don't know what caused it, but only this one error. Can you help me to have a look? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To finish up run
npm run prettier:write
and add a log to the CHANGELOG.
build stats
and coverage
tests fails because your branch is not part of fabric so that is fine. We need to fix that.
. |
@ShaMan123 Do I need to run |
yes, exactly |
@ShaMan123 ok, it's ready |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great job!
Thanks!
Motivation
Description
Make lint PASS!
Changes
var => let/const
eslint ignore
Gist
In Action