Temurin version incorrectly converted for setup-java 6.0.x
まだ誰も着手していません。
- 主要言語
- TypeScript
- スター
- 2k
- フォーク
- 870
- 平均マージ
- 3時間 16分
- マージ済み PR(30日)
- 11
説明
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
コントリビューションガイド
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
調査の方向性
Java バージョンを処理する正規表現がある src/distributions/base-installer.ts の 681~689 行付近から始めます。26.0.2.1+1 や 25.0.4.1+1 などの Temurin バージョンで失敗を再現し、その後、setup-java が SemVer エラーなしでこれらのバージョンを正常に解決できることを確認します。
索引モデルが issue の本文から書いたものです。
評価
- 技術スタック
- github-actions, java, typescript
- 領域
- ci-cd, devops
- issue の種類
- バグ
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 活発さ
- 活発
- 明瞭さ
- 明確に書かれている
- 初心者へのやさしさ
- 82/100