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

Initial proof of concept for expanding ameba's functionality with semantic information #550

Draft
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

nobodywasishere
Copy link
Contributor

@nobodywasishere nobodywasishere commented Jan 27, 2025

Some notes:

  • Entrypoints are currently not implemented, still work to do to figure out how to support that
  • Requires Crystal installed on the system it's running (previously this was not a requirement)
  • Only does top-level semantic (for performance) and is gated behind a --semantic flag
    • It may be worth changing the --semantic flag into an enum to choose between top-level and full semantic
    • Top-level was chosen for performance reasons

There are a lot of tests and improvements I want to make to this code but stopping for now to get feedback.

Related: #513

Still a lot of things to do and figure out, primarily around entrypoints
Requires `--semantic` flag
@nobodywasishere nobodywasishere changed the title Initial proof of concept for expanding ameba's functionality with semantic information (#513) Initial proof of concept for expanding ameba's functionality with semantic information Jan 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant