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
Add drop_unused_requests option to exclude unused interactions #946
Conversation
Review |
(Ran the tests and recorded #948.) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like this change, and seems to do what it sets out to do.
(One thing we often ask is "How do we teach this?", and perhaps that's a further question, for later. But this is a new configuration option, and its place is in the user-visible documentation, in some form.)
The test failures in the cucumber tests have been reported separately.
6c3480a
to
1700922
Compare
Fix for curb issue here: #949 |
1700922
to
b3b58c2
Compare
Depends on (and is rebased on) this PR, which we should merge first: #949 |
I've added feature documentation in |
@ryanseys Cool, now with the other PR merged, we could re-run the tests, and that gave just one failure, which seems like a fixable one. Good job! (I'll wait for you to make that pass.) |
bf38ee2
to
9448215
Compare
Co-authored-by: Olle Jonsson <olle.jonsson@gmail.com>
9448215
to
0b5dc5e
Compare
0b5dc5e
to
9d687b0
Compare
@olleolleolle Should be passing now. Please take another look. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me!
@olleolleolle any chance for an official release of this feature? I tried it briefly and seemd quite awesome! |
@klyonrad You may try the new 6.2.0 release at your leisure. |
This is a rebase of #769 so that we can hopefully merge it! :)