Skip to content

Conversation

artembilan
Copy link
Member

Related to: #10083

  • Add missed package-info.java to packages of the test-support module
  • Add @NullMarked and fix all the reported nullability failures
  • Move hashCode() and equals() in the OnlyOnceTrigger to the end of class for more logical class structure.

Related to: spring-projects#10083

* Add missed `package-info.java` to packages of the `test-support` module
* Add `@NullMarked` and fix all the reported nullability failures
* Move `hashCode()` and `equals()` in the `OnlyOnceTrigger` to the end of class for more logical class structure.
Copy link
Contributor

@cppwfs cppwfs left a comment

Choose a reason for hiding this comment

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

Looks good!

Just one question and a couple of nitpicks.

@@ -1,4 +0,0 @@
/**
Copy link
Contributor

Choose a reason for hiding this comment

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

Asking:
Why was this package-info removed?

Copy link
Member Author

Choose a reason for hiding this comment

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

There is no content in this mail package at all.
So, no need in this directory and its dangling package-info.

Looks like I have missed to mention that in the commit message 😄

@artembilan artembilan requested a review from cppwfs July 22, 2025 14:22
@cppwfs
Copy link
Contributor

cppwfs commented Jul 22, 2025

LGTM!

@cppwfs
Copy link
Contributor

cppwfs commented Jul 22, 2025

Rebased, Squashed Merged

Thanks for the great work!

@cppwfs cppwfs closed this Jul 22, 2025
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.

2 participants