Skip to content

Conversation

@splix
Copy link

@splix splix commented Sep 16, 2023

fix #101

Copy link
Owner

@eprbell eprbell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM: thanks for submitting the PR!

Copy link
Owner

@eprbell eprbell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I spoke too soon: there are a few unit test failures. Also the documentation failure is due to a dead link: I disabled the check for it, so if you rebase the code it should go away.

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.

Parameter 'crypto_fee' has zero value

2 participants