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

resolves #497 introduce a load function #541

Merged
merged 1 commit into from
Mar 6, 2022

Conversation

ggrossetie
Copy link
Member

@ggrossetie ggrossetie commented Mar 6, 2022

What I Did

  • Extract SkinnyTextDocument into util/document.ts
  • Introduce a load function in AsciidocParser that returns the Asciidoctor.Document and a list of IncludeEntry
  • Remove getMediaDir from AsciidocParser (was unused)
  • Remove baseDocumentIncludeItems instance variable on AsciidocParser (since the load function now returns this information)
  • Remove getDocumentInformation from the convertUsingJavascript

resolves #497

@ggrossetie ggrossetie requested a review from danyill March 6, 2022 11:41
@ggrossetie ggrossetie merged commit e96ac0b into asciidoctor:master Mar 6, 2022
@ggrossetie ggrossetie deleted the load-function branch March 6, 2022 13:01
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.

Introduce a load function in AsciidocParser
1 participant