-
Notifications
You must be signed in to change notification settings - Fork 623
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
Contain exactly more detail (#26) #3849
base: master
Are you sure you want to change the base?
Conversation
@AlexCue987 Should this be refactored to use the similarity package now? |
of course, that's the plan. This will add more info to the output. No code will be removed, a couple lines will be added. As this PR is large as is, do we want to make it slightly larger, or should we add similarity in a later PR? WDYT @Kantis ? |
so the enhanced message should look like this:
|
There are conflicts on the merge. |
just resolved a redeclaration. did I miss any other conflicts? |
this is the PR which adds search for similar elements in the same matcher: #3910 |
improve output of shouldContainExactly, try to find matching sublists, just like
git diff does
, as follows: