Skip to content
This repository was archived by the owner on Aug 21, 2025. It is now read-only.

fix tox build and tests - #20

Merged
thunderbiscuit merged 1 commit into
bitcoindevkit:masterfrom
kornpow:build/fix-tests
Aug 4, 2022
Merged

fix tox build and tests#20
thunderbiscuit merged 1 commit into
bitcoindevkit:masterfrom
kornpow:build/fix-tests

Conversation

@kornpow

@kornpow kornpow commented Jul 1, 2022

Copy link
Copy Markdown
Contributor

Signed-off-by: Sam Korn korn94sam@gmail.com

Note:

Description

Notes to the reviewers

Im a little confused about this line, and the requirement for junk

db_config = bdk.DatabaseConfig.MEMORY(junk="")

Checklists

All Submissions:

New Features:

  • I've added tests for the new feature
  • I've added docs for the new feature
  • I've updated CHANGELOG.md

@kornpow
kornpow force-pushed the build/fix-tests branch 2 times, most recently from 50d267c to 7d088eb Compare July 1, 2022 17:55
@thunderbiscuit

Copy link
Copy Markdown
Member

I think your PR is made against an older version of master. Mind rebasing? This will fix your requirement for junk because the library now uses bdk-ffi 0.7.0.

@kornpow
kornpow force-pushed the build/fix-tests branch 2 times, most recently from ae5b2f8 to 73678b3 Compare July 15, 2022 16:04
@kornpow

kornpow commented Jul 15, 2022

Copy link
Copy Markdown
Contributor Author

@thunderbiscuit Ive pulled in the 0.7.0 bdk-ffi and got things working

@kornpow
kornpow force-pushed the build/fix-tests branch 2 times, most recently from 5ea1fa3 to 24f5a41 Compare July 15, 2022 16:21
@kornpow

kornpow commented Jul 15, 2022

Copy link
Copy Markdown
Contributor Author

I was doing a few things to the tests, but it appears like master's tests are the most accurate so I just scrapped everything I did there.

@thunderbiscuit thunderbiscuit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looking good! I still have some questions regarding the manifest and the pyproject.toml but that's mostly because I don't have a ton of experience building Python packages as of yet. I followed up on Discord.

Comment thread MANIFEST.in Outdated
Comment thread tox.ini
Signed-off-by: Sam Korn <korn94sam@gmail.com>

@thunderbiscuit thunderbiscuit left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK 4396eba.

@thunderbiscuit
thunderbiscuit merged commit b19f776 into bitcoindevkit:master Aug 4, 2022
@thunderbiscuit thunderbiscuit mentioned this pull request Aug 5, 2022
@notmandatory notmandatory added this to the Release 0.2.0 milestone Aug 6, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

No open projects
Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants