jchen293 commented 1 year ago

Description

Clean up Java build warnings by:

Adding @InlineMe to the deprecated function so that when users call the method tagged with @InlineMe will be migrated to the new method. This implementation is optional but it's worth adding it.
Removing the printStackTrace() and throw an exception instead

Testing

E2E ran make clean publish-dry

Screenshot 2023-09-06 at 4 19 00 PM

Please select the option(s) that are relevant to this PR.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Improvement (fixing a typo, updating readme, renaming a variable name, etc)

Justintime50 commented 1 year ago

Please add a description to the summary about how this cleans up the errors. I'm not sure what the inline syntax does.

jchen293 commented 1 year ago

Please add a description to the summary about how this cleans up the errors. I'm not sure what the inline syntax does.

Something optional to add, updated the description of the PR summary