Temurin version incorrectly converted for setup-java 6.0.x
Dieses Issue hat noch niemand übernommen.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 2k
- Forks
- 870
- Ø Merge
- 3 Std. 16 Min.
- Gemergte PRs (30 T.)
- 11
Beschreibung
Description:
After upgrade from 5.7.0 to 6.0.0 and 6.0.1, I noticed that setup-java does not recognize Temurin's versions anymore (e.g. "26.0.2+10"). This leads to an "is not valid SemVer notation for a Java version" error.
The following regular expression does not not pick up Temurin's version correctly:
The issue seems to be similar to: https://github.com/actions/setup-java/issues/1267
Task version:
v6, v6.0.1
Platform:
- Ubuntu
- macOS
- Windows
Runner type:
- Hosted
- Self-hosted
Repro steps:
- successful run for setup-java "5.7.0" and Temurin "26.0.2+10", "25.0.4+7"
- failed runs for setup-java "6.0.0" and Temurin "26.0.2.1+1", "25.0.4.1+1"
- failed runs for setup-java "6.0.1" and Temurin "26.0.2.1+1", "25.0.4.1+1"
Expected behavior:
Successfully resolve Temurin version.
Actual behavior:
setup-java >= 6.0.0 returns
The string '26.0.2.1+1' is not valid SemVer notation for a Java version. Please check README file for code snippets and more detailed information
Beitragsleitfaden
Erste Schritte
- Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
- Forke das Repository und arbeite in einem Branch.
- Öffne einen Pull Request, der die Issue-Nummer nennt.
Rechercherichtung
Beginnen Sie in src/distributions/base-installer.ts etwa bei den Zeilen 681–689, wo der reguläre Ausdruck Java-Versionen verarbeitet. Reproduzieren Sie den Fehler mit Temurin-Versionen wie 26.0.2.1+1 und 25.0.4.1+1 und überprüfen Sie anschließend, dass setup-java diese Versionen erfolgreich auflöst, ohne den SemVer-Fehler.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Bewertung
- Tech-Stack
- github-actions, java, typescript
- Bereich
- ci-cd, devops
- Issue-Typ
- Bug
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Aktivitätsstatus
- Aktiv
- Klarheit
- Klar beschrieben
- Anfängerfreundlichkeit
- 82/100