Skip to content

AddressTest: use FieldSource and record for reading key/addr parameters#336

Open
msgilligan wants to merge 1 commit intomasterfrom
msgilligan/AddressTest-use-fieldsource-and-records
Open

AddressTest: use FieldSource and record for reading key/addr parameters#336
msgilligan wants to merge 1 commit intomasterfrom
msgilligan/AddressTest-use-fieldsource-and-records

Conversation

@msgilligan
Copy link
Copy Markdown
Member

@msgilligan msgilligan commented Apr 9, 2026

About AddressTest: This is a WIP test in a WIP module that aims to implement Taproot functionality with secp-jdk and bitcoinj.

The commit updated to FieldSource was cherry-picked/merged and the remaining commit uses a record instead of the Jupiter Arguments class.

@msgilligan msgilligan added this to the Version 0.2.1 milestone Apr 9, 2026
@msgilligan msgilligan force-pushed the msgilligan/AddressTest-use-fieldsource-and-records branch from 9345770 to c114dcb Compare April 10, 2026 22:15
@msgilligan msgilligan changed the title AddressTest: use MethodSource and record for reading key/addr parameters AddressTest: use FieldSource and record for reading key/addr parameters Apr 12, 2026
@msgilligan msgilligan force-pushed the msgilligan/AddressTest-use-fieldsource-and-records branch from c114dcb to 1ab0b0d Compare April 12, 2026 01:57
@schildbach
Copy link
Copy Markdown
Member

Approved 3360ce9.

@msgilligan msgilligan force-pushed the msgilligan/AddressTest-use-fieldsource-and-records branch from 1ab0b0d to d7da6b0 Compare April 14, 2026 15:33
@msgilligan msgilligan force-pushed the msgilligan/AddressTest-use-fieldsource-and-records branch from d7da6b0 to 3007fdb Compare April 14, 2026 15:38
@msgilligan
Copy link
Copy Markdown
Member Author

Approved 3360ce9.

Cherry-picked and rebased.

@msgilligan
Copy link
Copy Markdown
Member Author

msgilligan commented Apr 14, 2026

@schildbach I updated the PR description as it was trying to explain that this PR applies changes to a test/functionality that itself is incomplete, not that the PR is incomplete. The update makes things more clear.

@msgilligan msgilligan removed this from the Version 0.2.1 milestone Apr 14, 2026
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