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 valid charset in Content-Type in HTML output #83

Merged
merged 1 commit into from
Dec 31, 2022

Conversation

deivid-rodriguez
Copy link
Contributor

See ISO 10646:2017 etc.

This PR ports the fix at n-ronn/nronn#27 from @tnir to the ronn-ng repo.

See ISO 10646:2017 etc.

Signed-off-by: Takuya Noguchi <takninnovationresearch@gmail.com>
@apjanke
Copy link
Owner

apjanke commented Dec 31, 2022

Ha. Surprised I never noticed that. Thanks!

@apjanke apjanke merged commit e125abe into apjanke:master Dec 31, 2022
@apjanke apjanke self-assigned this Dec 31, 2022
@apjanke apjanke added the bug Something isn't working label Dec 31, 2022
@apjanke apjanke added this to the 0.10.1 milestone Dec 31, 2022
@apjanke
Copy link
Owner

apjanke commented Dec 31, 2022

Merged. I think this is about ready for an 0.10 release. I'll review the outstanding bugs and test results, and if it looks good, cut an 0.10.1 ronn-ng release this weekend.

Unless there are some other fixes or enhancements you're in a hurry to see in a release, @deivid-rodriguez ?

@deivid-rodriguez deivid-rodriguez deleted the fix-charset branch December 31, 2022 15:36
@apjanke apjanke mentioned this pull request Dec 31, 2022
7 tasks
@tnir
Copy link
Contributor

tnir commented Jan 1, 2023

Thanks @apjanke

@deivid-rodriguez
Copy link
Contributor Author

Unless there are some other fixes or enhancements you're in a hurry to see in a release, @deivid-rodriguez ?

Forgot to answer here, no, there's not anything that should block the release further from my side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants