Skip to content

ci: Improve nextest config - #56462

Closed
MrSubidubi wants to merge 1 commit into
mainfrom
nextest-config
Closed

ci: Improve nextest config#56462
MrSubidubi wants to merge 1 commit into
mainfrom
nextest-config

Conversation

@MrSubidubi

Copy link
Copy Markdown
Member

As noted in https://nexte.st/docs/configuration/reference/?h=list+wrapper#default-configuration, immediate-final is the recommended setting for failure-output for large test suites, as is definitely the case for this repository.

Furthermore, allow one flaky run for the ocasionally flaky tests that we do have. We will add some tracking for which tests are flaky at a later point.

Release Notes:

  • N/A

@MrSubidubi
MrSubidubi requested a review from osiewicz May 11, 2026 19:04
@cla-bot cla-bot Bot added the cla-signed The user has signed the Contributor License Agreement label May 11, 2026
@zed-community-bot zed-community-bot Bot added the staff Pull requests authored by a current member of Zed staff label May 11, 2026

@osiewicz osiewicz left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I recall that @SomeoneToIgnore had strong objections to immediate-final, as we've tried it before: #39643

(from private convo):

...I do not see the list of what's failed, and have to scroll for eternity for another failure.

@MrSubidubi

Copy link
Copy Markdown
Member Author

Yeah I think that is a valid point, hadn't thought of it like that. Optimally, there would be another variant.

Will close for the time being.

@MrSubidubi MrSubidubi closed this May 13, 2026
@MrSubidubi
MrSubidubi deleted the nextest-config branch May 13, 2026 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla-signed The user has signed the Contributor License Agreement staff Pull requests authored by a current member of Zed staff

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants