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(ci): use the line reporter for now #8605

Merged
merged 1 commit into from
Jun 13, 2023
Merged

Conversation

jtoar
Copy link
Contributor

@jtoar jtoar commented Jun 13, 2023

After my change to split up the smoke tests into their own steps, which required using Playwright's built-in way of starting dev servers (I detailed all of it here: #8449), I forgot to follow up with a change to its reporter so that we could see each dev servers' output. Here it is.

image

-- snip --

image

@jtoar jtoar added the release:chore This PR is a chore (means nothing for users) label Jun 13, 2023
@jtoar jtoar requested a review from Tobbe June 13, 2023 22:46
Copy link
Member

@Tobbe Tobbe left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jtoar jtoar merged commit 3be3691 into main Jun 13, 2023
25 checks passed
@jtoar jtoar deleted the ds-ci/use-line-reporter-for-now branch June 13, 2023 23:11
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Jun 13, 2023
@jtoar jtoar modified the milestones: next-release, v6.0.0 Jun 14, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants