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

Child Activity: Preprocessing and Move from traditional Parsing of Imports to tree-sitter based parsing. #253

Open
JayGhiya opened this issue Jan 6, 2025 — with Huly for GitHub · 3 comments
Assignees

Comments

Copy link
Member

JayGhiya commented Jan 6, 2025

  1. Do custom preprocessing on node.
  2. While doing parsing of imports move to tree sitter based parsing.
  3. insertion into neo4j.
  4. Push the json to local mounted volume as mentioned in configuration.

Huly®: UNOPL-294

@JayGhiya JayGhiya self-assigned this Jan 6, 2025
Copy link
Member Author

JayGhiya commented Jan 6, 2025

We will be removing dependency on isort and be dependent on ruff as of now for import sorting. as ruff does not support comment based parsing of imports so we would not be able to differentiate between system and external. but we will be able to figure out internal imports which we need right now. Also right now we use absolute import conversion from ruff on codebases to make sure we get the full path of internal import irrespective of how relative it is. but it is an unsafe modification on codebases. so we cant run on actual codebases on branch. So what we will do- We will run them as of now in temporal worker to not cause any ill effect on actual codebases.

Copy link
Member Author

JayGhiya commented Jan 6, 2025

Copy link
Member Author

JayGhiya commented Jan 6, 2025

cc: Vipin Shreyas Kumar

@JayGhiya JayGhiya changed the title Move from traditional Parsing of Imports to tree-sitter based parsing. Child Activity: Preprocessing and Move from traditional Parsing of Imports to tree-sitter based parsing. Jan 6, 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

No branches or pull requests

1 participant