Skip to content

Include an explicit seed value in the RSpec command#174

Merged
anthony-j-castro merged 1 commit into
CircleCI-Public:masterfrom
elia:patch-3
May 14, 2026
Merged

Include an explicit seed value in the RSpec command#174
anthony-j-castro merged 1 commit into
CircleCI-Public:masterfrom
elia:patch-3

Conversation

@elia
Copy link
Copy Markdown
Contributor

@elia elia commented May 7, 2026

This makes the rspec command easy to copy and paste locally for debugging flaky tests that depend on execution order.

See https://linuxsimply.com/bash-scripting-tutorial/operator/arithmetic-operators/random-number/#:~:text=1.%20Using%20“RANDOM”%20Environment%20Variable for the builtin bash $RANDOM variable.

This makes the `rspec` command easy to copy and paste locally for debugging flaky tests that depend on execution order.
@elia elia requested a review from a team as a code owner May 7, 2026 17:16
@elia
Copy link
Copy Markdown
Contributor Author

elia commented May 13, 2026

cc @marboledacci @bschrag620 anything else I can do to get this reviewed/merged?

Copy link
Copy Markdown
Contributor

@bschrag620 bschrag620 left a comment

Choose a reason for hiding this comment

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

Seems reasonable to me. ➕ , though I don't have approve permissions on the repo.

@elia
Copy link
Copy Markdown
Contributor Author

elia commented May 14, 2026

@bschrag620 thanks for the soft-approval, I though you were a member, I guess I'll wait for @marboledacci 🙏

@anthony-j-castro anthony-j-castro merged commit 90b3360 into CircleCI-Public:master May 14, 2026
1 of 2 checks passed
@anthony-j-castro
Copy link
Copy Markdown

@elia thanks for your contribution. We're running into some errors in our workflow that's blocking publishing a new version. Going to take a look at fixing those so there will be a delay before this change will be available.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants