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

Move to openrewrite #117

Open
wants to merge 10 commits into
base: main
Choose a base branch
from
Open

Move to openrewrite #117

wants to merge 10 commits into from

Conversation

jimbethancourt
Copy link
Collaborator

Move from JavaParser to OpenRewrite

JavaFqnCapturingVisitor captures all possible type name references when specified as a generic type parameter
Extract FqnCapturingVisitor to simplify implementation of multi-language visiting
Processing JavaType types instead of J types.  This allows for types to be processed via dynamic type binding
- Rendering large graphs (> 700 nodes + edges) on demand with Sigma.js, Graphology, graphlib, and graphlib-dot in the HTML report.  Renders using WebGL.
- Touched up type processing in JavaVariableVisitor
- Removed the use of JavaParser
Removed unneeded text from the report output (package names and excess JavaScript characters in DOT graph declaration)
- Added JavaClassDeclarationVisitor and pulled type processing up into the TypeProcessor interface and added corresponding unit test

- Composing the JavaVistor with the various visitors needed to capture all types in the codebase
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