Skip to content

Use nextest profiles instead of cfgs#4059

Closed
matheus23 wants to merge 1 commit intoFrando/netsimfrom
matheus23/patchbay
Closed

Use nextest profiles instead of cfgs#4059
matheus23 wants to merge 1 commit intoFrando/netsimfrom
matheus23/patchbay

Conversation

@matheus23
Copy link
Copy Markdown
Member

Removes the patchbay_test cfg in favor of a nextest profile called patchbay, and disabling patchbay by default in cargo nextest runs.

@matheus23 matheus23 requested a review from Frando March 30, 2026 10:33
@matheus23 matheus23 self-assigned this Mar 30, 2026
@matheus23 matheus23 mentioned this pull request Mar 30, 2026
3 tasks
@github-actions
Copy link
Copy Markdown

Documentation for this PR has been generated and is available at: https://n0-computer.github.io/iroh/pr/4059/docs/iroh/

Last updated: 2026-03-30T10:36:02Z

@n0bot n0bot bot added this to iroh Mar 30, 2026
@github-project-automation github-project-automation bot moved this to 🚑 Needs Triage in iroh Mar 30, 2026
@github-actions
Copy link
Copy Markdown

Netsim report & logs for this PR have been generated and is available at: LOGS
This report will remain available for 3 days.

Last updated for commit: e308fa9

Copy link
Copy Markdown
Contributor

@flub flub left a comment

Choose a reason for hiding this comment

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

i like this

Comment thread .config/nextest.toml
threads-required = 32

[profile.default]
default-filter = 'not binary(patchbay)'
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What's the tradeoff between profile.default and profile.default.overrides?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think they do different things.
The [[profile.default.overrides]] specifies a filter for which tests to apply the override to, not a filter for which tests to run by default.

Frando added a commit that referenced this pull request Mar 30, 2026
@Frando
Copy link
Copy Markdown
Member

Frando commented Mar 30, 2026

Thanks! I merged this onto #3986 manually (had some conflicts because I did some similar changes myself, so hand-merged it).

@Frando Frando closed this Mar 30, 2026
@github-project-automation github-project-automation bot moved this from 🚑 Needs Triage to ✅ Done in iroh Mar 30, 2026
Frando added a commit that referenced this pull request Mar 30, 2026
Frando added a commit that referenced this pull request Mar 30, 2026
Frando added a commit that referenced this pull request Mar 31, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

3 participants