kpm - not setting plugin version should possible in PluginNamingResolver
まだ誰も着手していません。
評価
調査の方向性
まず PluginNamingResolver.java、特にそのコンストラクター、getPluginVersion()、静的ファクトリを確認し、次に参照されている呼び出しの周辺にある ArtifactAndVersionFinder.java を調べます。stringContainsVersion がどのように使用されているかを追跡し、存在する場合は既存の resolver テストを確認します。pluginVersion が省略されていてもプラグイン名から解析でき、解決できないバージョンが IllegalStateException を発生させずに指定されたフォールバックに従えば完了です。
索引モデルが issue の本文から書いたものです。
説明
If PluginNamingResolver created without specifying the pluginVersion, getPluginVersion() throw IllegalStateException. It is not correct, since we have a chances to analyzing version from stringContainsVersion.
IIRC, that assertion added when working with ArtifactAndVersionFinder, but as workaround, we can change that line in ArtifactAndVersionFinder with:
final String pluginNameAsArtifactId = pluginKey + "-plugin";
And then, we can remove PluginNamingResolver.of(pluginKey) static factory. And start to re-condider getPluginVersion() implementation. After all, none of operation in PluginNamingResolver should throw an exception, since even if we cannot found any version, it will return 0.0.0.
- 主要言語
- Java
- スター
- 30
- フォーク
- 42
- 平均マージ
- 2日 14時間
- マージ済み PR（30日）
- 2
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
killbill/killbill-platform のほかの issue
-
bug
難易度 5/5 1週間以上 初心者へのやさしさ 35/100
killbill/killbill-platform#193 ·
-
LogsSseHandler: allow caller to filter logs by accountId and userToken再び着手できるかも @xsalefter が 125 日前に担当しましたが、オープン中のプルリクエストはありません。 オープンenhancement
killbill/killbill-platform#163 · 担当者 1 名 ·
-
難易度 4/5 3〜5日 初心者へのやさしさ 35/100
killbill/killbill-platform#160 · コメント 1 件 ·
-
難易度 3/5 1〜2日 初心者へのやさしさ 45/100
-
難易度 3/5 1〜2日 初心者へのやさしさ 42/100
killbill/killbill-platform の issue をすべて見る
似ている issue
-
enhancement triage
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
ShaftHQ/SHAFT_ENGINE#7902 ·
メンテナーはふだん 1 日以内に返信
-
jules
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
open-learning-exchange/myplanet#18568 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
quarkiverse/quarkus-ironjacamar#264 · コメント 7 件 ·
-
[Bug]: Login shows only "Log-in failed" when Commons answers the login request with an API errorオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
commons-app/apps-android-commons#7295 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信