Skip to content

docs(reporters): clarify dot is default on CI#9074

Merged
mxschmitt merged 1 commit into
microsoft:masterfrom
anishkny:patch-2
Oct 4, 2021
Merged

docs(reporters): clarify dot is default on CI#9074
mxschmitt merged 1 commit into
microsoft:masterfrom
anishkny:patch-2

Conversation

@anishkny
Copy link
Copy Markdown
Contributor

@anishkny anishkny commented Sep 22, 2021

It is unclear from docs that CI uses dot reporter by default:

const defaultReporter: BuiltInReporter = process.env.CI ? 'dot' : 'list';

Maintainers: Please feel free to tweak wording as appropriate.

@anishkny anishkny changed the title docs(reporters): Clarify dot is default on CI docs(reporters): clarify dot is default on CI Sep 22, 2021
@mxschmitt mxschmitt merged commit 92c1aa2 into microsoft:master Oct 4, 2021
sidharthv96 added a commit to sidharthv96/playwright that referenced this pull request Oct 4, 2021
* upstream/master:
  feat(chromium): roll to r927453 (microsoft#9294)
  chore: fix reuseBrowserContextStorage spelling typo (microsoft#9296)
  test: remove checking duration in github reporter to reduce flakiness
  docs: fix wrongly annotated JS only paragraph (microsoft#9290)
  docs: add tabs for shell scripts (microsoft#9289)
  docs(reporters): clarify dot reporter is default on CI (microsoft#9074)
  browser(webkit): roll to 2021/09/30 (microsoft#9288)
  browser(chromium): roll to r927453 (microsoft#9285)
  types(test-runner): Add github to built in reporters (microsoft#9283)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants