Skip to content

Conversation

@mriise
Copy link
Contributor

@mriise mriise commented Nov 21, 2025

fixes #100

Copy link
Member

@vmx vmx left a comment

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.

@vmx
Copy link
Member

vmx commented Nov 22, 2025

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

@mriise
Copy link
Contributor Author

mriise commented Nov 23, 2025

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
Copy link
Contributor Author

mriise commented Nov 23, 2025

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

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

3 participants