Very small change. I removed this constraint in a previous commit as part of #322 thinking it's redundant. But that's not correct since most use cases don't care about choosing a strategy when building transactions. So most use sites will just require a verbose Default (TxBuilderStrategy m) constraint on top. This improves upon that.
Type of Change
Please mark the relevant option(s) for your pull request:
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Code refactoring (improving code quality without changing its behavior)
[ ] Documentation update (adding or updating documentation related to the project)
Checklist
Please ensure that your pull request meets the following criteria:
Summary
Very small change. I removed this constraint in a previous commit as part of #322 thinking it's redundant. But that's not correct since most use cases don't care about choosing a strategy when building transactions. So most use sites will just require a verbose
Default (TxBuilderStrategy m)
constraint on top. This improves upon that.Type of Change
Please mark the relevant option(s) for your pull request:
Checklist
Please ensure that your pull request meets the following criteria:
Testing
Additional Information