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

Integrating the new parser #425

Merged
merged 14 commits into from
Nov 26, 2023
Merged

Integrating the new parser #425

merged 14 commits into from
Nov 26, 2023

Conversation

SuperFola
Copy link
Member

New parser, new syntax

Description

This PR aims at integrating the parser made there into ArkScript v4. The tests for the parser have been added to the project, and code has been refactored (specifically in the compiler, to unify the code base and have a better integration with this new parser).

Closes #423

Checklist

  • I have read the Contributor guide
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have updated the documentation if needed
  • I have added tests that prove my fix/feature is working
  • New and existing tests pass locally with my changes

Former-commit-id: f567af034611eac9e999e2ebf9217e8b5d6d34ed
Former-commit-id: 1a6ce9b448487ca1f67284230c7f2e303cc4bac1
@SuperFola SuperFola added ❗️ priority/medium 💥 breaking change Indicate that the issue/PR is bringing breaking change(s) in the language labels Jun 7, 2023
@SuperFola SuperFola self-assigned this Jun 7, 2023
@github-actions github-actions bot added 👷 ci Related to our Continous Integration system (build, tests, auto-release...) ark/compiler Related to the compiler ark/macros Related to macros in ArkScript and to the macro processor ark/parser Parser related ark/repl Related to the REPL ark/virtual machine Related to the ark virtual machine labels Jun 7, 2023
@SuperFola SuperFola force-pushed the feat/parser branch 2 times, most recently from 154fb0f to fa7e3d2 Compare June 8, 2023 15:28
SuperFola added 5 commits June 8, 2023 17:43
…unding it in the compiler side of things

This new parser isn't ready to use yet, but if I continue, this commit will be 50000 lines by itself, while I just need something that can compile basic files and work the same way as before.
I still need to add an import solver, clean a lot of todo/fixme, (re)implement conditional macros, add the parser's tests in the CI, and some more cleanup, but we are getting there!
@SuperFola SuperFola force-pushed the feat/parser branch 9 times, most recently from f00b21d to bee077a Compare June 8, 2023 18:03
@SuperFola
Copy link
Member Author

This PR is dropping the (imo) useless support for utf8 identifiers ; however utf8 strings are still supported.

@SuperFola SuperFola force-pushed the feat/parser branch 20 times, most recently from 733f1f7 to 8f472d2 Compare November 26, 2023 16:16
@SuperFola SuperFola merged commit 078da45 into dev Nov 26, 2023
35 checks passed
@SuperFola SuperFola deleted the feat/parser branch November 26, 2023 17:13
@ArkScript-lang ArkScript-lang deleted a comment from notion-workspace bot Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ark/compiler Related to the compiler ark/macros Related to macros in ArkScript and to the macro processor ark/parser Parser related ark/repl Related to the REPL ark/virtual machine Related to the ark virtual machine 💥 breaking change Indicate that the issue/PR is bringing breaking change(s) in the language 👷 ci Related to our Continous Integration system (build, tests, auto-release...) ❗️ priority/medium 🧪 tests Related to the project tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

v4: integrate the new parser
1 participant