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
Add 'object-curly-newline' eslint rule. #5523
Conversation
✅ Tests for the commit de6e1ce have passed. See details: |
src/api/test-controller/index.js
Outdated
MultipleWindowsModeIsDisabledError, | ||
MultipleWindowsModeIsNotAvailableInRemoteBrowserError | ||
} from '../../errors/test-run'; | ||
import { MultipleWindowsModeIsDisabledError, MultipleWindowsModeIsNotAvailableInRemoteBrowserError } from '../../errors/test-run'; |
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.
Maybe we can enable consistent: true
to allow separate long imports even when their number is below the critical value
❌ Tests for the commit 4f7f61b have failed. See details: |
@testcafe-build-bot retest |
❌ Tests for the commit 4f7f61b have failed. See details: |
4f7f61b
to
68a73b1
Compare
❌ Tests for the commit 68a73b1 have failed. See details: |
✅ Tests for the commit 68a73b1 have passed. See details: |
FPR |
#5391 (comment)
The object-curly-newline affects the line breaks only inside of the statement (not after). However, I guess we can add this rule to standardize the new lines inside of
import
statements.The new lines are required if there are:
I added a special rule for tests because the statement
import { Selector, Role, t } from 'testcafe'
with 3 items looks good.