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

feat: parsing let statements #18

Merged
merged 11 commits into from
Apr 7, 2024
Merged

feat: parsing let statements #18

merged 11 commits into from
Apr 7, 2024

Conversation

katherinerbanis
Copy link
Collaborator

@katherinerbanis katherinerbanis commented Apr 2, 2024

Created the ast.go package which contains the base node, statement node, and expression node interfaces as well as struct and tokenLiteral().

Also created the parser.go and parser_test.go. Parser imports the ast.go, lexer.go, and token.go packages, contains a struct, token functions, statement functions, error functions, and began the ParseProgram function.

Closes #14.

BabyKangaroo117 and others added 7 commits March 20, 2024 18:41
Because we don't have our package published / setup to publish properly, we can't pull a package from the github domain.
Created ast package which contains base, statement, expression node interfaces.
Also created parser package which imports the ast, lexer, and token packages, contains token functions, statement functions, peek functions, has an error function, and began the ParseProgram() function. Created parser_test.go package as well.
@dineshUmasankar dineshUmasankar removed a link to an issue Apr 6, 2024
4 tasks
@dineshUmasankar dineshUmasankar changed the title parser and ast packages Chapter 2.1 - 2.4 Parsing Let Statements Apr 6, 2024
@dineshUmasankar dineshUmasankar changed the title Chapter 2.1 - 2.4 Parsing Let Statements feat: parsing let statements Apr 6, 2024
@dineshUmasankar dineshUmasankar self-requested a review April 6, 2024 23:24
@dineshUmasankar dineshUmasankar added the enhancement New feature or request label Apr 6, 2024
Copy link
Owner

@dineshUmasankar dineshUmasankar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

From here on out, I would appreciate some inline documentation for every method and type that is defined and a package-level comment to describe the intent / purpose of the package. @katherinerbanis

@dineshUmasankar dineshUmasankar merged commit 06ee0fa into main Apr 7, 2024
@dineshUmasankar dineshUmasankar deleted the Katherine-Parser branch April 7, 2024 02:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Chapters 2.1 - 2.4 (Parse Let Statements)
3 participants