Skip to content

feat: base45#116

Merged
vmx merged 3 commits into
masterfrom
base45
Dec 17, 2025
Merged

feat: base45#116
vmx merged 3 commits into
masterfrom
base45

Conversation

@mriise

@mriise mriise commented Nov 21, 2025

Copy link
Copy Markdown
Contributor

fixes #100

@vmx vmx 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.

Long time no see @mriise! Thanks for the PR.

Comment thread tests/lib.rs
@vmx

vmx commented Nov 22, 2025

Copy link
Copy Markdown
Member

I first thought the no_std failures were unrelated. But it looks like the base45 is not no_std compatible :-/

@mriise

mriise commented Nov 23, 2025

Copy link
Copy Markdown
Contributor Author

It just needs to feature flag alloc.

I was sort of hoping base45 wasn't on the case insensitive list, the crate assumes it to be uppercase (because that's what the rfc says).
This comment is weird but understandable I guess. multiformats/multibase#125 (comment)

I'll just toUpper() before handing it over to the crate. (oh god the camel case has infected me)
:\

@mriise

mriise commented Nov 23, 2025

Copy link
Copy Markdown
Contributor Author

@vmx unless we have a user waiting on this, im ok with waiting a bit for no_std to be merged upstream before this gets merged.

@mriise

mriise commented Dec 16, 2025

Copy link
Copy Markdown
Contributor Author

merged and updated :)

@vmx vmx 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.

Thanks!

@vmx vmx merged commit f035d95 into master Dec 17, 2025
10 checks passed
@vmx vmx deleted the base45 branch December 17, 2025 22:13
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.

Add base45

2 participants