chore: remove references to legacy issue trackers

Author: triceo

core/src/main/java/ai/timefold/solver/core/config/localsearch/LocalSearchType.java

@@ -26,7 +26,6 @@ public enum LocalSearchType {
      */
     public static @NonNull LocalSearchType @NonNull [] getBluePrintTypes() {
         return Arrays.stream(values())
-                // Workaround for https://issues.redhat.com/browse/PLANNER-1294
                 .filter(localSearchType -> localSearchType != SIMULATED_ANNEALING)
                 .toArray(LocalSearchType[]::new);
     }

core/src/main/java/ai/timefold/solver/core/impl/heuristic/selector/list/RandomSubListSelector.java

@@ -106,7 +106,6 @@ public long getValueCount() {
 
     @Override
     public Iterator<SubList> iterator() {
-        // TODO make this incremental https://issues.redhat.com/browse/PLANNER-2507
         var biggestListSize = 0;
         for (var entity : ((Iterable<Object>) entitySelector::endingIterator)) {
             biggestListSize = Math.max(biggestListSize, listVariableDescriptor.getUnpinnedSubListSize(entity));

core/src/main/java/ai/timefold/solver/core/impl/score/director/incremental/IncrementalScoreDirector.java

@@ -154,8 +154,6 @@ public void afterVariableChanged(VariableDescriptor<Solution_> variableDescripto
         super.afterVariableChanged(variableDescriptor, entity);
     }
 
-    // TODO Add support for list variable (https://issues.redhat.com/browse/PLANNER-2711).
-
     @Override
     public void beforeListVariableElementAssigned(ListVariableDescriptor<Solution_> variableDescriptor, Object element) {
         incrementalScoreCalculator.beforeListVariableElementAssigned(variableDescriptor.getVariableName(), element);

core/src/main/java/ai/timefold/solver/core/impl/util/CollectionUtils.java

@@ -28,7 +28,6 @@ public static <E> List<E> copy(List<E> originalList, boolean reverse) {
         /*
          * Some move implementations on the hot path rely heavily on list reversal.
          * As such, the following implementation was benchmarked to perform as well as possible for lists of all sizes.
-         * See PLANNER-2808 for details.
          */
         switch (originalList.size()) {
             case 0 -> {

core/src/test/java/ai/timefold/solver/core/impl/score/stream/bavet/BavetAdvancedGroupByConstraintStreamTest.java

@@ -558,7 +558,7 @@ void filteredFromUniquePair() {
         InnerScoreDirector<TestdataLavishSolution, SimpleScore> scoreDirector =
                 buildScoreDirector(factory -> factory.forEachUniquePair(TestdataLavishEntity.class,
                         Joiners.equal(TestdataLavishEntity::getEntityGroup),
-                        Joiners.filtering((e1, e2) -> !e1.getCode().contains("My"))) // Filtering() caused PLANNER-2139.
+                        Joiners.filtering((e1, e2) -> !e1.getCode().contains("My")))
                         .penalize(SimpleScore.ONE)
                         .asConstraint(TEST_CONSTRAINT_NAME));
 
@@ -575,7 +575,7 @@ void filteredFromUniquePair() {
     }
 
     @TestTemplate
-    void groupByThenJoinThenGroupBy() { // PLANNER-2270
+    void groupByThenJoinThenGroupBy() {
         assertThatCode(() -> buildScoreDirector(factory -> factory.forEach(TestdataLavishEntity.class)
                 .groupBy(TestdataLavishEntity::getEntityGroup, TestdataLavishEntity::getValue)
                 .join(TestdataLavishEntity.class)

core/src/test/java/ai/timefold/solver/core/impl/score/stream/collector/bi/InnerBiConstraintCollectorsTest.java

@@ -656,7 +656,7 @@ public void toMap() {
     }
 
     @Test
-    public void toMapDuplicates() { // PLANNER-2271
+    public void toMapDuplicates() {
         BiConstraintCollector<String, Integer, ?, Map<String, Set<Integer>>> collector =
                 ConstraintCollectors.toMap((a, b) -> a, (a, b) -> b);
         Object container = collector.supplier().get();

core/src/test/java/ai/timefold/solver/core/impl/score/stream/common/AbstractAdvancedGroupByConstraintStreamTest.java

@@ -557,7 +557,7 @@ void filteredFromUniquePair() {
         InnerScoreDirector<TestdataLavishSolution, SimpleScore> scoreDirector =
                 buildScoreDirector(factory -> factory.forEachUniquePair(TestdataLavishEntity.class,
                         equal(TestdataLavishEntity::getEntityGroup),
-                        filtering((e1, e2) -> !e1.getCode().contains("My"))) // Filtering() caused PLANNER-2139.
+                        filtering((e1, e2) -> !e1.getCode().contains("My")))
                         .penalize(SimpleScore.ONE)
                         .asConstraint(TEST_CONSTRAINT_NAME));
 
@@ -574,7 +574,7 @@ void filteredFromUniquePair() {
     }
 
     @TestTemplate
-    void groupByThenJoinThenGroupBy() { // PLANNER-2270
+    void groupByThenJoinThenGroupBy() {
         assertThatCode(() -> buildScoreDirector(factory -> factory.forEach(TestdataLavishEntity.class)
                 .groupBy(TestdataLavishEntity::getEntityGroup, TestdataLavishEntity::getValue)
                 .join(TestdataLavishEntity.class)
@@ -586,7 +586,7 @@ void groupByThenJoinThenGroupBy() { // PLANNER-2270
     }
 
     @TestTemplate
-    void reusedStreamsInJoin() { // PLANNER-2884
+    void reusedStreamsInJoin() {
         TestdataLavishSolution solution = TestdataLavishSolution.generateSolution(1, 1, 2, 4);
 
         ConstraintProvider cp = constraintFactory -> {

tools/benchmark/src/main/java/ai/timefold/solver/benchmark/impl/statistic/StatisticUtils.java

@@ -55,14 +55,13 @@ public static double[] determineStandardDeviationDoubles(
         return standardDeviationDoubles;
     }
 
-    // TODO Do the locale formatting in benchmarkReport.html.ftl - https://issues.redhat.com/browse/PLANNER-169
     public static String getStandardDeviationString(double[] standardDeviationDoubles) {
         if (standardDeviationDoubles == null) {
             return null;
         }
         StringBuilder standardDeviationString = new StringBuilder(standardDeviationDoubles.length * 9);
         // Abbreviate to 2 decimals
-        // We don't use a local sensitive DecimalFormat, because other Scores don't use it either (see PLANNER-169)
+        // We don't use a local sensitive DecimalFormat, because other Scores don't use it either
         DecimalFormatSymbols decimalFormatSymbols = new DecimalFormatSymbols(Locale.US);
         DecimalFormat exponentialFormat = new DecimalFormat("0.0#E0", decimalFormatSymbols);
         DecimalFormat decimalFormat = new DecimalFormat("0.0#", decimalFormatSymbols);