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.

Sign up for GitHub

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

Jump to bottom

Use (List|Set).of() where feasible #3231

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Use (List|Set).of() where feasible #3231

wants to merge 1 commit into from

Conversation

quaff
Copy link
Contributor

@quaff quaff commented Jan 8, 2025

No description provided.

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jan 8, 2025
@mp911de mp911de added type: task A general task and removed status: waiting-for-triage An issue we've not yet triaged labels Jan 13, 2025
@mp911de mp911de self-assigned this Jan 13, 2025
@kse-music
Copy link

kse-music commented Jan 17, 2025 •
edited
Loading

Note that Arrays.asList is mutable and List.of is not. Switching to an immutable Collection would be considered as a breaking change.

@quaff
Copy link
Contributor Author

quaff commented Jan 17, 2025

Note that Arrays.asList is mutable and List.of is not. Switching to an immutable Collection would be considered as a breaking change.

So I didn't replace all Arrays.asList().

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@mp911de mp911de

Labels
type: task A general task
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@quaff @kse-music @mp911de @spring-projects-issues