This is an automated email from the ASF dual-hosted git repository.
jamesbognar pushed a commit to branch release-manager
in repository https://gitbox.apache.org/repos/asf/juneau.git
The following commit(s) were added to refs/heads/release-manager by this push:
new 0a8d3dd328 Use Shorts.cp() for defensive list copy in MilestoneService
0a8d3dd328 is described below
commit 0a8d3dd3284ea900c04b4868cb06e131bba1de8b
Author: James Bognar <[email protected]>
AuthorDate: Wed Aug 19 13:08:30 2026 -0400
Use Shorts.cp() for defensive list copy in MilestoneService
Adopt juneau's Shorts.cp() collection alias instead of ArrayList
construction when copying the incoming PR list before sorting.
---
.../java/org/apache/juneau/releng/milestone/MilestoneService.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/src/main/java/org/apache/juneau/releng/milestone/MilestoneService.java
b/src/main/java/org/apache/juneau/releng/milestone/MilestoneService.java
index dbb11a5435..079733602e 100644
--- a/src/main/java/org/apache/juneau/releng/milestone/MilestoneService.java
+++ b/src/main/java/org/apache/juneau/releng/milestone/MilestoneService.java
@@ -17,6 +17,8 @@
package org.apache.juneau.releng.milestone;
+import static org.apache.juneau.commons.utils.Shorts.*;
+
import java.util.ArrayList;
import java.util.LinkedHashMap;
import java.util.List;
@@ -39,7 +41,7 @@ public class MilestoneService {
/** Groups Dependabot bump PRs into one {@link ChangelogEntry} per
dependency, sorted by dependency. */
public List<ChangelogEntry> generateChanges(List<PullRequest> prs) {
// Preserve encounter order within a group, but iterate PRs by
ascending number for stable from/to + refs.
- var sorted = new ArrayList<>(prs);
+ var sorted = cp(prs);
sorted.sort((a, b) -> Integer.compare(a.number, b.number));
var groups = new LinkedHashMap<String,
ChangelogEntry.Builder>();