Skip to content
This repository has been archived by the owner on May 20, 2024. It is now read-only.

blankline after multiline expression #10

Merged
merged 1 commit into from
Jun 18, 2019
Merged

Conversation

kuceb
Copy link
Contributor

@kuceb kuceb commented Jun 17, 2019

examples of what this fixes:

image

image

image

@chrisbreiding
Copy link
Collaborator

Can you post an example of what this does?

Does it mean the following:

const foo = {
  bar: 'baz',
}
const bar = 'baz'

will have to be the following?

const foo = {
  bar: 'baz',
}

const bar = 'baz'

@kuceb
Copy link
Contributor Author

kuceb commented Jun 17, 2019

no, it means:

sinon
.stub(fs, 'pathExistsAsync')
.withArgs(binaryPkgPath)
.resolves(true)
sinon
.stub(fs, 'readJsonAsync')
.withArgs(binaryPkgPath)
.resolves({ version: '2.0.48' })

will have to be

sinon
.stub(fs, 'pathExistsAsync')
.withArgs(binaryPkgPath)
.resolves(true)

sinon
.stub(fs, 'readJsonAsync')
.withArgs(binaryPkgPath)
.resolves({ version: '2.0.48' })

@kuceb
Copy link
Contributor Author

kuceb commented Jun 17, 2019

also will add newlines between it/describe blocks

it('foo', () => {
})

it('bar', () => {
})

@brian-mann
Copy link
Member

Ohhh this is perfect. Exactly what we were looking for.

@brian-mann brian-mann merged commit d351766 into cypress-io:master Jun 18, 2019
@kuceb kuceb mentioned this pull request Jun 18, 2019
panzarino pushed a commit to cypress-io/cypress that referenced this pull request Sep 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants