Skip to content
This repository has been archived by the owner on Oct 1, 2018. It is now read-only.

Revisit parser architecture #114

Closed
5 tasks done
zdne opened this issue Jun 3, 2014 · 2 comments · Fixed by #203
Closed
5 tasks done

Revisit parser architecture #114

zdne opened this issue Jun 3, 2014 · 2 comments · Fixed by #203
Assignees

Comments

@zdne
Copy link
Contributor

zdne commented Jun 3, 2014

Revisit parser architecture.

Improve the parser architecture so it is easier to introduce new syntax (particularly list-based section) and maintain the older one.

This includes:

  • Separate Markdown parser from Snow Crash
  • Use Markdown AST (tree) instead of a flat stack of blocks
  • Provide uniform interface for processing markdown sections regardless of the definition type (markdown header or list)
  • Revisit work with source maps (range conversations)
  • Simplify unit tests – No need to construct Markdown AST in tests – parse Markdown / Blueprint directly
@zdne zdne self-assigned this Jun 3, 2014
@zdne
Copy link
Contributor Author

zdne commented Jun 3, 2014

New Parser Architecture Proof of Concept

https://gist.github.com/zdne/55b7a38f52ca642e123b

Markdown Parser

https://github.com/apiaryio/markdown-parser

@zdne zdne assigned pksunkara and unassigned zdne Jul 24, 2014
@zdne
Copy link
Contributor Author

zdne commented Jul 24, 2014

Progress-tracking Wiki page: https://github.com/apiaryio/snowcrash/wiki/Refactor-Progress

@pksunkara pksunkara added this to the Architecture Refactor milestone Jul 24, 2014
@zdne zdne closed this as completed in #203 Sep 18, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants