Skip to content

Conversation

@arxanas
Copy link
Owner

@arxanas arxanas commented Feb 3, 2023

No description provided.

@arxanas arxanas force-pushed the arxanas/test branch 3 times, most recently from 430f121 to ccb297a Compare February 3, 2023 07:53
@arxanas arxanas changed the title feat(test): create search module feat(test): create scm-bisect crate Feb 3, 2023
@arxanas arxanas force-pushed the arxanas/test branch 6 times, most recently from 3a5a690 to f4bcdf9 Compare February 9, 2023 23:05
@arxanas arxanas changed the title feat(test): create scm-bisect crate feat(test): create git test run --search option Feb 9, 2023
@arxanas arxanas marked this pull request as ready for review February 9, 2023 23:21
@arxanas arxanas force-pushed the arxanas/test branch 2 times, most recently from 81f4793 to 071244a Compare February 10, 2023 05:06
Fortunately, no bugs were detected at this stage. I was able to detect a previous bug I had committed when artificially reintroducing it.
This way, we can also use the caller's trait and lifetime bounds. This was necessary because the calling code previously didn't know that the contained `dyn Error` was `Send`.
- At the end of the search, the caller wants to get the search bounds without caring about `next_to_search`.
- In the future, we may have different kinds of bounds. For example, we could track bounds where *any* monotonic function (with respect to ancestry on the DAG) changes value, instead of just boolean success and failure.
@arxanas arxanas merged commit a5deb6c into master Feb 10, 2023
@arxanas arxanas deleted the arxanas/test branch February 10, 2023 07:59
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