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

Merge AST extractor and source extractor #18

Open
pauxus opened this issue Jun 6, 2024 · 1 comment
Open

Merge AST extractor and source extractor #18

pauxus opened this issue Jun 6, 2024 · 1 comment

Comments

@pauxus
Copy link
Contributor

pauxus commented Jun 6, 2024

Both extract javadoc from an AST node, so source extraction should just be a fallback step from AST extraction.

Refactor separate steps into services, which can be injected (SPI mechanism)

@pauxus
Copy link
Contributor Author

pauxus commented Jun 7, 2024

This is actually not necessary. Since the source-extractor runs in an earlier compilation phase, when the AST extractor is used, the javadoc will already be placed in an annotation.

Still the naming should be improved.

@pauxus pauxus changed the title Merge AST extractor an source extractor Merge AST extractor and source extractor Jun 16, 2024
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