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

Parser TS bindings #7881

Merged
merged 48 commits into from
Oct 11, 2023
Merged

Parser TS bindings #7881

merged 48 commits into from
Oct 11, 2023

Conversation

kazcw
Copy link
Contributor

@kazcw kazcw commented Sep 24, 2023

Pull Request Description

Generate TS bindings and lazy deserialization for the parser types.

Important Notes

  • The new API is imported into ffi.ts, but not yet used.
  • I have tested the generated code in isolation, but cannot commit tests as we are not currently able to load WASM modules when running in vitest.

Checklist

Please ensure that the following checklist has been satisfied before submitting the PR:

  • The documentation has been updated, if necessary.
  • Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
  • All code follows the
    Scala,
    Java,
    and
    Rust
    style guides. In case you are using a language not listed above, follow the Rust style guide.
  • All code has been tested:
    • Unit tests have been written where possible.
    • If GUI codebase was changed, the GUI was tested when built using ./run ide build.

@kazcw kazcw self-assigned this Sep 24, 2023
@kazcw kazcw added the CI: No changelog needed Do not require a changelog entry for this PR. label Sep 26, 2023
@kazcw kazcw marked this pull request as ready for review September 29, 2023 05:31
@kazcw kazcw closed this Oct 4, 2023
@kazcw kazcw reopened this Oct 4, 2023
@Frizi Frizi added the -gui label Oct 9, 2023
@kazcw kazcw marked this pull request as ready for review October 10, 2023 16:00
@Frizi Frizi added the CI: Ready to merge This PR is eligible for automatic merge label Oct 11, 2023
@mergify mergify bot merged commit 9fd1ab9 into develop Oct 11, 2023
34 checks passed
@mergify mergify bot deleted the wip/kw/parser-ts branch October 11, 2023 13:04
mergify bot pushed a commit that referenced this pull request Oct 11, 2023
Removes dependencies that should not be in the root `package.json`
Basic testing done with `npm run ci-check`.

Introduced by the following PRs (in order of being merged):
See blame for more details: https://github.com/enso-org/enso/blame/develop/package.json
- #7813
- #7656 (oops)
- #7881

# Important Notes
None
mergify bot pushed a commit that referenced this pull request Oct 19, 2023
- Validate spans during existing lexer and parser unit tests, and in `enso_parser_debug`.
- Fix lost span info causing failures of updated tests.

# Important Notes
- [x] Output of `parse_all_enso_files.sh` is unchanged since before #7881 (modulo libs changes since then).
- When the parser encounters an input with the first line indented, it now creates a sub-block for lines at than indent level, and emits a syntax error (every indented block must have a parent).
- When the parser encounters a number with a base but no digits (e.g. `0x`), it now emits a `Number` with `None` in the digits field rather than a 0-length digits token.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
-gui CI: Clean build required CI runners will be cleaned before and after this PR is built. CI: No changelog needed Do not require a changelog entry for this PR. CI: Ready to merge This PR is eligible for automatic merge
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Exposing AST API through typescript
4 participants