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

Support standard markdown titles #44

Merged
merged 1 commit into from
Jan 8, 2021
Merged

Support standard markdown titles #44

merged 1 commit into from
Jan 8, 2021

Conversation

leebyron
Copy link
Owner

@leebyron leebyron commented Jan 8, 2021

Supports use of standard markdown headers (# Header) as the title of a spec markdown document. Inspired by a similar change in #31 however this does not require wrapping the header in bold.

For some context, originally setext headers were required to differentiate between a document title and a section header in an imported file since spec-md did not differentiate between initial files and imported files. This change is possible by having pegjs support multiple starting positions for the parser and parse initial and imported documents differently.

This also allows us to remove the ast validation logic since that can now be encoded directly in the grammar.

Supports use of standard markdown headers (`# Header`) as the title of a spec markdown document. Inspired by a similar change in #31 however this does not require wrapping the header in bold.

For some context, originally setext headers were required to differentiate between a document title and a section header in an imported file since spec-md did not differentiate between initial files and imported files. This change is possible by having pegjs support multiple starting positions for the parser and parse initial and imported documents differently.

This also allows us to remove the ast validation logic since that can now be encoded directly in the grammar.
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.

None yet

1 participant