Skip to content

Fill changelog with current changes - #4225

Open
Earlopain wants to merge 3 commits into
ruby:mainfrom
Earlopain:pending-changelog
Open

Earlopain wants to merge 3 commits into
ruby:mainfrom
Earlopain:pending-changelog

Conversation

@Earlopain

Copy link
Copy Markdown
Collaborator

Might need to add more if stuff gets changed

Comment thread CHANGELOG.md
## [Unreleased]

Highlights of this release:
- Significantly improved parsing performance by up to 40%.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on the generated node.rb

Might need to add more if stuff gets changed
Comment thread CHANGELOG.md Outdated

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

LGTM

Earlopain and others added 2 commits September 11, 2026 11:31
Co-authored-by: Benoit Daloze <eregontp@gmail.com>
@Earlopain

Copy link
Copy Markdown
Collaborator Author

I'll just leave this open until the point when the release actually happens

@kddnewton

Copy link
Copy Markdown
Collaborator

To make this a little easier for consumers, I'd prefer to have specific sections for the different bindings. For example, the JS bindings changed a lot by accepting a UInt8Array at the main entrypoint, C changed a lot because it now uses uint32_t for locations, and Ruby has far fewer changes. So, I think we should split it up.

@kddnewton kddnewton left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Marking as request changes

@Earlopain

Copy link
Copy Markdown
Collaborator Author

Sure, I'll add sections for this later

This branch has not been deployed

No deployments
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.

3 participants