Skip to content

Normalize native extension naming - #26

Merged
ioquatix merged 1 commit into
mainfrom
normalize-native-extension-naming
Aug 20, 2026
Merged

Normalize native extension naming#26
ioquatix merged 1 commit into
mainfrom
normalize-native-extension-naming

Conversation

@samuel-williams-shopify

Copy link
Copy Markdown
Contributor

Summary

  • Rename Markly's native wrapper identifiers to mirror their Ruby constant paths, following the conventions used by io-event.
  • Name typed-data descriptors and callbacks consistently as Markly_Node_Type, Markly_Node_Type_mark, and related forms.
  • Use conventional suffixes such as _allocate and _p where appropriate.
  • Group class registration into Init_Markly_Node and Init_Markly_Parser helpers.

This is an internal naming and organization refactor; it does not change the Ruby API.

Testing

  • Built the native extension with ./extconf.rb && make -j4.
  • Ran a native smoke test covering extension discovery, parser construction, table parsing and rendering, Node#fence, and Node.new.

@ioquatix
ioquatix merged commit eaeb0d2 into main Aug 20, 2026
22 of 28 checks passed
@ioquatix
ioquatix deleted the normalize-native-extension-naming branch August 20, 2026 07:00
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