Alpha and beta releases sort above the final release
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
ソートは src/Repository/ReleasesCollection.php の comparisonVersionString() にあります。この関数は -beta.1 形式のサフィックスを数値部分に書き換えますが、その結果を version_compare() が誤って順位付けしてしまいます。まず ReleasesCollectionTest::sortByVersionPutsPreReleasesBelowFinalRelease から #[Skip] を削除し、vendor/bin/testo --filter=sortByVersionPutsPreReleasesBelowFinalRelease を実行してください。このテストが通り、正式リリースが rc、beta、alpha より上に並ぶようになり、大文字の -RC も含めて正しく扱われれば完了です。リリース検索を置き換えるとこのコードが削除される可能性があるため、先に #49 を確認してください。
索引モデルが issue の本文から書いたものです。
説明
Description
ReleasesCollection::sortByVersion() ranks alpha and beta pre-releases above the final release of the same version. Sorting v2024.1.0-alpha.1, v2024.1.0, v2024.1.0-beta.1, v2024.1.0-rc.1 gives:
v2024.1.0-beta.1
v2024.1.0-alpha.1
v2024.1.0
v2024.1.0-rc.1
Expected: v2024.1.0, v2024.1.0-rc.1, v2024.1.0-beta.1, v2024.1.0-alpha.1.
get-binary and download-protoc-binary install the first release of the sorted list that passes the stability filter. With --stability=beta (or lower) they pick the beta over the final release that is already out.
comparisonVersionString() replaces -<stability> with .<weight>.. v2024.1.0-beta.1 therefore compares as v2024.1.0.2..1, and version_compare() ranks it above v2024.1.0 because it has more numeric parts. Lower-case -rc tags are left untouched (the stability is RC), so they happen to sort correctly. Upper-case -RC tags have the same problem as beta and alpha. #2 was about the order among pre-releases.
Steps to reproduce
- Remove
#[Skip]fromReleasesCollectionTest::sortByVersionPutsPreReleasesBelowFinalRelease. - Run
vendor/bin/testo --filter=sortByVersionPutsPreReleasesBelowFinalRelease.
Additional context
The release lookup may be replaced by DLoad in #49, in which case this code goes away.
- 主要言語
- PHP
- スター
- 30
- フォーク
- 15
- 平均マージ
- 45分
- マージ済み PR（30日）
- 11
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
roadrunner-php/cli のほかの issue
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
roadrunner-php/cli#69 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
roadrunner-php/cli#68 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
roadrunner-php/cli#67 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
roadrunner-php/cli#62 ·
メンテナーはふだん 1 日以内に返信
-
bug
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
roadrunner-php/cli#61 ·
メンテナーはふだん 1 日以内に返信
roadrunner-php/cli の issue をすべて見る
似ている issue
-
No Code Attached Yet
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
joomla/joomla-cms#48611 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
Bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
Automattic/safe-publish#716 ·
メンテナーはふだん 1 日以内に返信
-
[FrameworkBundle][SecurityBundle] Make `KernelBrowser::loginUser()` wor…対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープンFrameworkBundle SecurityBundle
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
symfony/symfony-docs#23277 ·
メンテナーはふだん 3 日以内に返信
-
[Sync EN] hard link does not need special permissions対応中かも @lacatoire が今日担当しました。 オープンsync-en
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信