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

tech debt cleanup #1986

Merged
merged 14 commits into from
Aug 12, 2019
Merged

tech debt cleanup #1986

merged 14 commits into from
Aug 12, 2019

Conversation

ssalinas
Copy link
Member

@ssalinas ssalinas commented Jul 19, 2019

Figured right after a release is a good time to do this since we have a new snapshot version to start working off of. Currently includes:

  • guava -> java util Optional
  • remove Throwables.propagate
  • clean up various other deprecated methods

@ssalinas ssalinas changed the title WIP - tech debt cleanup Tech debt cleanup Aug 2, 2019
@ssalinas
Copy link
Member Author

ssalinas commented Aug 2, 2019

Note for posterity, ObjectMapper was updated to NOT_ABSENT setting. GuavaModule had this behavior by default but jdk8 module does not

@ssalinas ssalinas added the hs_qa label Aug 7, 2019
@ssalinas ssalinas changed the title Tech debt cleanup tech debt cleanup Aug 7, 2019
@baconmania
Copy link
Contributor

🚢

@ssalinas ssalinas merged commit 82c26bc into master Aug 12, 2019
@ssalinas ssalinas deleted the tech_debt branch August 12, 2019 14:33
@ssalinas ssalinas added this to the 0.24.0 milestone Aug 12, 2019
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.

2 participants