Skip to content
This repository has been archived by the owner on Nov 23, 2021. It is now read-only.

Review and potentially rewrite annotation processor in the bb-annotations module #299

Closed
mkrzyzanowski opened this issue Nov 6, 2018 · 1 comment
Closed

Review and potentially rewrite annotation processor in the bb-annotations module #299

mkrzyzanowski opened this issue Nov 6, 2018 · 1 comment
Assignees
@mkrzyzanowski
Labels
bug help wanted
Milestone
2.0#3

Comments

@mkrzyzanowski
Copy link
Contributor

SonarQube raises Critical bugs related to APIs from sun.* packages used in our annotation processor and required system-scoped dependencies.

Review them and potentially rewrite the whole annotation processing.

Do we actually need the stuff it provides, i.e. injected currentScope field?
@mkrzyzanowski mkrzyzanowski added this to the 2.0#3 milestone Nov 6, 2018
@mkrzyzanowski mkrzyzanowski self-assigned this Nov 27, 2018
mkrzyzanowski added a commit that referenced this issue Nov 28, 2018
@mkrzyzanowski
#299 Removed bb-annotations module
b355792
@mkrzyzanowski
Copy link
Contributor Author

Annotation processor was removed in the end.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@mkrzyzanowski mkrzyzanowski

Labels
bug help wanted
Projects
None yet
Milestone
2.0#3
Development

No branches or pull requests
1 participant
@mkrzyzanowski