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

#156977411 Implement Parser #10

Closed
wants to merge 268 commits into from

Conversation

appcypher
Copy link
Collaborator

@appcypher appcypher commented May 27, 2018

What does this PR do?

This PR implements a packrat parser and it is expected to be faster than the old parser.

Description of Task to be completed?

  • Create a packrat parser.
  • Reduce boilerplate stuff by having a basic parser combinator built in.
  • Write tests for parser.
  • Add a custom deep-equal implementation for tests.
  • Improve testing library; add for support travis and coverage report.

How should this be manually tested?

  • Install necessary packages with npm i.
  • Run parser test, node src/compiler/syntax/parser-tests.js.
  • The parser.js file is self-contained, it can be imported and used in other files.

What are the relevant pivotal tracker stories?

156977411

Project Story

https://github.com/astrolang/astro/projects/1#card-12436138

@appcypher appcypher changed the title #156977411 Implement Parser #156977411 Implement Parser [WIP - Don't Merge!] May 27, 2018
@appcypher appcypher changed the title #156977411 Implement Parser [WIP - Don't Merge!] #156977411 Implement Parser Aug 7, 2018
@appcypher appcypher closed this Sep 12, 2018
@appcypher appcypher deleted the ch-implement-packrat-parser-156977411 branch September 12, 2018 22:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant