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: enable object-curly-spacing in ESLint #9168

Merged

Conversation

@mxschmitt
Copy link
Member

@mxschmitt mxschmitt commented Sep 27, 2021

Visual Studio Code does automatically do this when formatting code. In our source-code we have a big mixture between with spaces, without spaces, or sometimes even a mix out of both. When people format it with VSC this leads to fix then other's source-code pieces and so a weird diff gets created.

@mxschmitt mxschmitt force-pushed the linting-enable-object-curly-spacing branch from aa56f9a to 015c4bd Sep 27, 2021
@mxschmitt mxschmitt merged commit cd22072 into microsoft:master Sep 27, 2021
9 checks passed
sidharthv96 added a commit to sidharthv96/playwright that referenced this issue Sep 28, 2021
* upstream/master: (26 commits)
  browser(webkit): disable COOP support (microsoft#9185)
  fix(test runner): proper serial mode with beforeAll/afterAll failures (microsoft#9183)
  chore: remove browserType.connect from .net - not yet ready (microsoft#9182)
  test: canvas updates are reflected on screenshots (microsoft#9180)
  chore: dedupe return types in the dotnet api generator (microsoft#9181)
  docs: mention empty string in `userDataDir` (microsoft#9069)
  feat(expect): toContainText(array) (microsoft#9160)
  chore: enable object-curly-spacing in ESLint (microsoft#9168)
  feat(chromium): roll to r925110 (microsoft#9175)
  chore: render expect in trace viewer (microsoft#9141)
  devops: use Node.js 16 when rolling browsers
  docs(python): add docs about threading (microsoft#8829)
  browser(chromium): roll to r925110 (microsoft#9171)
  chore(test-runner): launch -> webServer (microsoft#9167)
  docs: cleanup test.describe.parallel.only doc (microsoft#9159)
  chore: remove FatalDOMError (microsoft#9119)
  feat(test runner): improve fixture typings for function fixtures (microsoft#9138)
  fix(expect): produce "waiting for selector" log, corner cases (microsoft#9140)
  fix: increase recent logs buffer (microsoft#9143)
  test: unflake a few tests (microsoft#9142)
  ...
sidharthv96 added a commit to sidharthv96/playwright that referenced this issue Sep 28, 2021
…sAnnotations

* upstream/master:
  browser(webkit): disable COOP support (microsoft#9185)
  fix(test runner): proper serial mode with beforeAll/afterAll failures (microsoft#9183)
  chore: remove browserType.connect from .net - not yet ready (microsoft#9182)
  test: canvas updates are reflected on screenshots (microsoft#9180)
  chore: dedupe return types in the dotnet api generator (microsoft#9181)
  docs: mention empty string in `userDataDir` (microsoft#9069)
  feat(expect): toContainText(array) (microsoft#9160)
  chore: enable object-curly-spacing in ESLint (microsoft#9168)
  feat(chromium): roll to r925110 (microsoft#9175)
  chore: render expect in trace viewer (microsoft#9141)
  devops: use Node.js 16 when rolling browsers
  docs(python): add docs about threading (microsoft#8829)
  browser(chromium): roll to r925110 (microsoft#9171)
  chore(test-runner): launch -> webServer (microsoft#9167)
  docs: cleanup test.describe.parallel.only doc (microsoft#9159)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants