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

Add CollectionUtils.isNotEmpty() method #34519

Closed
wants to merge 2 commits into from

Conversation

darckyn
Copy link

@darckyn darckyn commented Feb 28, 2025

The isNotEmpty method was created to making the code more readable and standardized.

Thank you.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged or decided on label Feb 28, 2025
Signed-off-by: Rodrigo Darti <[email protected]>
Signed-off-by: Rodrigo Darti <[email protected]>
Signed-off-by: Rodrigo Darti <[email protected]>
@sbrannen sbrannen added in: core Issues in core modules (aop, beans, core, context, expression) status: declined A suggestion or change that we don't feel we should currently apply and removed status: waiting-for-triage An issue we've not yet triaged or decided on labels Feb 28, 2025
@sbrannen
Copy link
Member

This request has already been declined multiple times.

In light of that, I am declining this PR as well.

@sbrannen sbrannen closed this Feb 28, 2025
@darckyn
Copy link
Author

darckyn commented Feb 28, 2025

It's a shame.

If there are already several requests for these methods, it's because there is a demand for standardization and use of the methods.

Libraries like Apache StringUtils use these methods and are very useful in development.

Now we'll wait a little longer for you to reject another request from other developers asking for the implementation of these methods.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
in: core Issues in core modules (aop, beans, core, context, expression) status: declined A suggestion or change that we don't feel we should currently apply
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants