Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rust Wallet #39

Merged
merged 200 commits into from
Oct 4, 2024
Merged

Rust Wallet #39

merged 200 commits into from
Oct 4, 2024

Conversation

opendansor
Copy link

Create a rust version of the wallet.

basfroman and others added 23 commits October 3, 2024 00:45
Replaced the usage of `PyUserWarning` with a custom `KeyFileError` for more specific error handling. This change promotes better clarity and consistency in error reporting within the `keyfile.rs` module.
Corrected the phrasing of status messages in keyfile.rs from "Encryption data..." and "Decrypt data..." to "Encrypting data..." and "Decrypting data...". This enhances clarity and consistency in the user interface.
* Remove except usage

* use alias for now

* use different constructor format

* fix log

* remove extra import

* fix keypair.rs errors

* chore: ruff

* chore: clippy

* chore: fmt
* replace with pyproject.toml

* fix release for manylinux builds

* rename package and file

* fix classifiers

* remove btwallet dir

* make non pre-release

* try ignoring tests dir

* try with maturin config

* add bip39

* test with more platforms

* remove other platforms

* comment out other platforms
@ibraheem-abe ibraheem-abe marked this pull request as ready for review October 4, 2024 00:46
@ibraheem-abe ibraheem-abe requested a review from a team October 4, 2024 00:46
@opendansor opendansor merged commit 94a0f67 into main Oct 4, 2024
8 of 11 checks passed
@opendansor opendansor self-assigned this Oct 4, 2024
@camfairchild camfairchild mentioned this pull request Oct 4, 2024
3 tasks
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.

6 participants