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

Clean up annotations #208

Merged
merged 4 commits into from
May 7, 2018
Merged

Clean up annotations #208

merged 4 commits into from
May 7, 2018

Conversation

ZacSweers
Copy link
Collaborator

This cleans up a few annotations areas for a few reasons.

  • Switch to internal DoNotMock annotation, as error-prone removed theirs. We're leaving in our own annotation for documentation purposes and any custom error prone checks that still look for annotations with this name.
  • Remove javaxextras for compatibility with the Java 9 module system, which unfortunately JSR 305 is not compatible with.
  • Remove stale jsr305 dependency.

This cleans up a few annotations areas for a few reasons.

- Switch to internal `DoNotMock` annotation, as error-prone removed theirs. We're leaving in our own annotation for documentation purposes and any custom error prone checks that still look for annotations with this name.
- Remove javaxextras for compatibility with the Java 9 module system, which unfortunately JSR 305 is not compatible with.
- Remove stale jsr305 dependency.
@ZacSweers ZacSweers added this to the 0.8.0 milestone May 7, 2018
@ZacSweers ZacSweers self-assigned this May 7, 2018
@ZacSweers ZacSweers requested review from msridhar and kageiit May 7, 2018 21:12
@ZacSweers ZacSweers merged commit 5bd31c5 into master May 7, 2018
@ZacSweers ZacSweers deleted the z/removeDoNotMock branch May 7, 2018 22:41
@ZacSweers ZacSweers mentioned this pull request May 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants