This repository was archived by the owner on Nov 30, 2024. It is now read-only.
Load spec files in the order that files and dirs were specified in the CLI. #2253
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
I believe this should be an acceptable compromise between filesystem consistency and running files as expected, if you need the files within folders in a certain order you'd have to specify them as such but thats just a case of being explicit. Nice work @myronmarston |
|
This is so much fun :) It's a killer feature for me. I can create a list of files to "order" and prepend them on the command line - and those will be run first. I can have another file of "slow" files - and append them to the command line. I can also prepend failures and/or common failures. Tons of cool options have opened up for me - so thanks a million! ❤️ |
myronmarston
force-pushed
the
myron/load-spec-files-in-cli-order
branch
from
3f2b5dd to
b6c84bc
Compare
|
Deduplication works great :) Can't wait to use this on a more heavier project tomorrow! |
mvidner
added a commit
to yast/yast-rake
that referenced
this pull request
Jul 22, 2016
to undo the nondeterminism introduced in rspec-3.5.0 rspec/rspec-core#2253
MatheusRich
pushed a commit
to MatheusRich/rspec-core
that referenced
this pull request
Oct 30, 2020
…-order Load spec files in the order that files and dirs were specified in the CLI.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This fixes #2247 and addresses some of the feedback from #2251.