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

use runes #299

Closed
wants to merge 4 commits into from
Closed

use runes #299

wants to merge 4 commits into from

Conversation

alovak
Copy link
Contributor

@alovak alovak commented Nov 8, 2023

No description provided.

@cheukwing
Copy link
Contributor

Thank you, this is much simpler than my fix!
We should also fix padding/right.go, and add some unit tests to the padders to finish off this draft.

@alovak
Copy link
Contributor Author

alovak commented Nov 9, 2023

@cheukwing do you have a capacity to work on this feature taking this PR as a base? You better know what it should do as I first time meet the need to use utf8 :)

@cheukwing
Copy link
Contributor

I will have some capacity in a couple of weeks, but I've also asked some of my team members if they could help out too 🙂

@alovak alovak closed this Feb 16, 2024
@alovak
Copy link
Contributor Author

alovak commented Feb 16, 2024

I don't see any good options to support utf8 for now

@alovak alovak reopened this Feb 16, 2024
@alovak
Copy link
Contributor Author

alovak commented Jun 11, 2024

I think that with new new opton to setup custom Packer / Unpacker (#310), we can handle utf8 / latin if / when we need without this PR.

@alovak alovak closed this Jun 11, 2024
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.

2 participants