-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
[eslint] enable object-curly-spacing and no-global-assign #9486
[eslint] enable object-curly-spacing and no-global-assign #9486
Conversation
317ebaa
to
94f9131
Compare
94f9131
to
edf7f2c
Compare
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.
LGTM
[ 6, true, '192.168.0.1', 'php', 'Linux', 40.141592, 'bat' ], | ||
[ 7, true, '192.168.0.1', 'php', 'Linux', 40.141592, 'bat' ], | ||
[ 8, true, '192.168.0.1', 'php', 'Linux', 40.141592, 'bat' ], | ||
[ 9, true, '192.168.0.1', 'php', 'Linux', 40.141592, 'bat' ], |
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.
Nice solution! At first I was like 😖 but then I was like 😮 and now I'm like 😏
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.
hahahahaha
@elastic/eslint-config-kibana
src/fixtures/hits.js
so that it complies with the max-len rule and is more similar tosrc/fixtures/logstash_fields.js
note: the majority of the changes were automated, the first and last commit were manual