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

Replace elements-fun with rust-elements - #134

Merged
luckysori merged 1 commit into
masterfrom
no-need-for-fun
Feb 16, 2021
Merged

luckysori merged 1 commit into
masterfrom
no-need-for-fun

Conversation

@luckysori

Copy link
Copy Markdown
Collaborator

Introduces a patch dependency on luckysori/rust-elements until ElementsProject/rust-elements#70 is merged.

@luckysori luckysori self-assigned this Feb 5, 2021

@thomaseizinger thomaseizinger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You will need to remove the fuzz CI job :)

@luckysori

Copy link
Copy Markdown
Collaborator Author

You will need to remove the fuzz CI job :)

Quick catch, thanks!

Comment thread waves/wallet/build.rs
Comment thread waves/wallet/src/typed_js_future.rs Outdated
Comment thread .github/workflows/ci.yml
Comment thread waves/wallet/src/wallet.rs Outdated
@luckysori
luckysori force-pushed the no-need-for-fun branch 2 times, most recently from db9fd23 to 56ddb53 Compare February 15, 2021 23:42

@thomaseizinger thomaseizinger left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Good to go in from my side! Thank you for making this effort @luckysori !

Comment thread elements-harness/src/elementd_rpc.rs Outdated
Comment thread swap/tests/happy_path.rs Outdated
Comment thread waves/wallet/src/wallet/coin_selection.rs Outdated
Introduces a patch dependency on `luckysori/rust-elements` until
ElementsProject/rust-elements#70 is merged.
@luckysori
luckysori merged commit c9d001c into master Feb 16, 2021
@luckysori
luckysori deleted the no-need-for-fun branch February 16, 2021 03:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants