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

fix: add missing @deprecated and @since to javadoc #210

Merged
merged 5 commits into from
Mar 21, 2025

Conversation

dwalluck
Copy link
Contributor

Add some missing javadoc.

Fix some method visibility issues and remove use of deprecated methods from code.

@dwalluck dwalluck force-pushed the deprecated-and-since branch from 6bf1488 to d5ae5d6 Compare March 17, 2025 22:12
@dwalluck dwalluck force-pushed the deprecated-and-since branch 3 times, most recently from 09795a0 to 66887d9 Compare March 17, 2025 22:45
@dwalluck dwalluck force-pushed the deprecated-and-since branch 2 times, most recently from f5f06c9 to 9c4cf57 Compare March 18, 2025 18:27
@dwalluck dwalluck changed the title fix: Add missing @deprecated and @since to javadoc fix: add missing @deprecated and @since to javadoc Mar 18, 2025
@dwalluck dwalluck force-pushed the deprecated-and-since branch 3 times, most recently from 09d7a99 to 0d2ba8f Compare March 18, 2025 18:51
jeremylong
jeremylong previously approved these changes Mar 19, 2025
Copy link
Collaborator

@jeremylong jeremylong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

jeremylong
jeremylong previously approved these changes Mar 19, 2025
Copy link
Collaborator

@jeremylong jeremylong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jeremylong
Copy link
Collaborator

@ppkarwasz you good with this one now?

@ppkarwasz
Copy link
Contributor

Now that Spotless is there, there will be some conflicts. This should help:

git checkout origin/master -- pom.xml
mvn spotless:apply

Add some missing javadoc.

Fix some method visibility issues and remove use of deprecated methods
from code.
@dwalluck dwalluck force-pushed the deprecated-and-since branch from 1d4ad9b to 72aea80 Compare March 19, 2025 13:29
@ppkarwasz
Copy link
Contributor

@dwalluck,

Could you avoid forced pushes to PRs and use merge instead? With merges, you can incrementally follow the changes to the PR, with forced pushes you always need to review it from the beginning.

@dwalluck
Copy link
Contributor Author

Could you avoid forced pushes to PRs and use merge instead? With merges, you can incrementally follow the changes to the PR, with forced pushes you always need to review it from the beginning.

I understand., I tended to be worried about a bunch of messages in the history for projects that don't squash before they merge.

jeremylong
jeremylong previously approved these changes Mar 21, 2025
@jeremylong
Copy link
Collaborator

@dwalluck mind fixing the merge conflicts? thanks!

@jeremylong jeremylong merged commit f3762e6 into package-url:master Mar 21, 2025
3 checks passed
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.

3 participants