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

CLI: Add --seed=true to generate a new seed #1800

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Krinkle
Copy link
Member

@Krinkle Krinkle commented Sep 16, 2024

  • Change --seed to require a value.
    This fixes the conflict with test file arguments, raised in CLI --seed without value and files argument is counterintuitive #1691.
  • Add --seed=true to generate a new seed.
    This makes it consistent with URL Params, where ?seed (which translated to seed=true) already did this. With this patch, we now also support pre-config qunit_config_seed=true and qunit_config_seed="true". As well as --seed=true on the CLI. And URL param ?seed=true (in case someone tries/expects that).

While at it, this also:

  • Remove duplicate logic from CLI for generating new seed value in value of /src/core/config.js, by translating --seed to a preconfig.

Fixes #1691.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

CLI --seed without value and files argument is counterintuitive
1 participant