Skip to content

Refactor Parser and Parser.Builder - #452

Closed
ani28-bit wants to merge 2 commits into
commonmark:mainfrom
ani28-bit:refactor-parser
Closed

ani28-bit wants to merge 2 commits into
commonmark:mainfrom
ani28-bit:refactor-parser

Conversation

@ani28-bit

Copy link
Copy Markdown

Summary

This PR refactors the parser implementation to improve readability and maintainability without changing functionality.

Changes

  • Refactored the duplicated parsing workflow in Parser.
  • Extracted extension handling, validation, and inline parser factory logic from Parser.Builder.

Verification

  • All existing tests pass successfully with mvn test.
  • No external behavior or API changes were introduced.

@robinst

robinst commented Oct 10, 2026

Copy link
Copy Markdown
Collaborator

Thanks. I like some of these changes but don't think others are an improvement at all; closing this.

@robinst robinst closed this Oct 10, 2026
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