-
Notifications
You must be signed in to change notification settings - Fork 706
SONARJAVA-5829 Add internal API for accessing classpath information #5341
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
SONARJAVA-5829 Add internal API for accessing classpath information #5341
Conversation
a424511 to
a2a71be
Compare
| * This is an extension that is instantiated per-project (per-module in Maven terms) and can be injected when needed. | ||
| */ | ||
| @ScannerSide | ||
| public interface ClasspathResolver { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I was thinking about adding the @org.sonar.java.annotations.Beta annotation here, it's an explicit way to say "use with care, not fully supported"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In an earlier version, I made a copy of @Beta and called it @Internal, to not only say that the API might change, but also to say that it is not supposed to be accessed by Sensors outside of the SonarSource org.
Does the @Beta annotation have any practical effect beyond documentation purposes?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think it's for documentation only, I like the idea of having @Internal, we could go for that
|
|
This PR is stale because it has been open 7 days with no activity. If there is no activity in the next 7 days it will be closed automatically |
|
Closed in favor of SONARJAVA-5845 |





SONARJAVA-5829
WIP API design, not final.