Skip to content

Conversation

@xokdvium
Copy link
Contributor

@xokdvium xokdvium commented Nov 9, 2025

…hanges

Motivation

I've run into this quite a few times when working with characterization test infra. It would print an invalid command:

_NIX_TEST_ACCEPT=1 meson test main/lang

Which you'd then proceed to run and it would fail. This commit makes it be honest about the command you need to run:

_NIX_TEST_ACCEPT=1 meson test --suite main lang

Context


Add 👍 to pull requests you find important.

The Nix maintainer team uses a GitHub project board to schedule and track reviews.

…hanges

I've run into this quite a few times when working with characterization test
infra. It would print an invalid command:

_NIX_TEST_ACCEPT=1 meson test main/lang

Which you'd then proceed to run and it would fail. This commit makes it
be honest about the command you need to run:

_NIX_TEST_ACCEPT=1 meson test --suite main lang
@xokdvium xokdvium requested a review from Ericson2314 November 9, 2025 13:56
@xokdvium xokdvium requested a review from edolstra as a code owner November 9, 2025 13:56
@github-actions github-actions bot added the with-tests Issues related to testing. PRs with tests have some priority label Nov 9, 2025
@Ericson2314
Copy link
Member

Nice catch :)

@Ericson2314 Ericson2314 added this pull request to the merge queue Nov 9, 2025
Merged via the queue into master with commit 18941cb Nov 9, 2025
20 checks passed
@Ericson2314 Ericson2314 deleted the honest-characterization-message branch November 9, 2025 17:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

with-tests Issues related to testing. PRs with tests have some priority

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants