RepositoriesCollection drops releases of all but the last repository
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
調査の方向性
src/Repository/RepositoriesCollection.php の getReleases() から始めてください。ここではジェネレータが yield from を使い、Collection::create() が iterator_to_array() を実行しているため、衝突する整数キーが保持されます。RepositoriesCollectionTest::mergesReleasesOfAllRepositories から #[Skip] を削除し、vendor/bin/testo --filter=mergesReleasesOfAllRepositories を実行して、まず失敗を確認してください。テストが成功し、すべてのリポジトリのリリースが結果に含まれていれば完了です。
索引モデルが issue の本文から書いたものです。
説明
Description
RepositoriesCollection::getReleases() loses releases when it aggregates more than one repository. Two repositories with one release each produce a collection with only the second release.
getReleases() builds the collection from a generator that uses yield from over each repository's releases. Collection::create() turns it into an array with iterator_to_array(), which keeps the keys. Every repository yields keys starting at 0, so a later repository overwrites the releases of an earlier one at the same index.
Today Command::getRepository() passes a single GitHub repository, so the CLI is not affected yet. Any second source added to the collection would silently replace the first one's releases.
Steps to reproduce
- Remove
#[Skip]fromRepositoriesCollectionTest::mergesReleasesOfAllRepositories. - Run
vendor/bin/testo --filter=mergesReleasesOfAllRepositories.
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 日以内に返信