[Java]: Refactor test for better use of matrix
メンテナーはふだん 1 日以内に返信
@edburns がすでに取り組んでいます。
2026年8月10日 から。
評価
この issue はまだ評価されていません。
説明
Predecessor: #1917
Successor: #2577
Problem
The java-sdk-tests.yml workflow has two separate jobs — java-sdk and java-sdk-inprocess — that duplicate ~20 lines of identical setup (checkout, setup-java, setup-node). This duplication makes maintenance harder: action version pin bumps must be applied in two places.
Proposal
Add a transport: [default, inprocess] dimension to the existing test-jdk matrix in the java-sdk job, and remove the standalone java-sdk-inprocess job.
Matrix shape
strategy:
fail-fast: false
matrix:
test-jdk: ["25", "17"]
transport: ["default", "inprocess"]
exclude:
- test-jdk: "17"
transport: "inprocess"
The exclude prevents the JDK-17 + inprocess combination, which is not meaningful (the native library targets a single JDK).
Conditional step adjustments
- Steps already guarded with
if: matrix.test-jdk == '25'gain an additional&& matrix.transport == 'default'where they should not run for the inprocess leg (e.g., spotless, javadoc, doc-version test). - The
mvninvocation switches based onmatrix.transport:mvn clean verify -Pinprocessfor inprocess, the existing split build-then-test flow for default. - Artifact names and test report titles include the transport value for clarity.
Benefits
- Less duplication — shared setup steps defined once.
- Easier maintenance — action pin bumps in one place.
- Consistent naming — job names like
Java SDK Tests (JDK 25, inprocess)clearly identify the configuration.
Risks
- Slightly more complex
if:conditionals, but manageable given the existing pattern.
- 主要言語
- TypeScript
- スター
- 10.5k
- フォーク
- 1.5k
- 平均マージ
- 1日 7時間
- マージ済み PR（30日）
- 64
環境構築
このプロジェクトの開発コンテナを、あなたの GitHub アカウントでブラウザ上に起動します。
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
github/copilot-sdk のほかの issue
-
Clarify SDK architecture and in-process runtime transport対応中かも @KalebCole が 4 日前に担当しました。 オープンdocumentation
難易度 1/5 1時間未満 初心者へのやさしさ 92/100
github/copilot-sdk#2804 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Python ModelLimits drops max_output_tokens from model metadata対応中かも @HDMowri が 5 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
github/copilot-sdk#2798 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
github/copilot-sdk#2793 ·
メンテナーはふだん 1 日以内に返信
-
agentic-workflows
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
github/copilot-sdk#2782 ·
メンテナーはふだん 1 日以内に返信
-
Rust: subagent lifecycle hooks are logged as unknown対応中かも @hackberry-lab が 8 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
github/copilot-sdk#2781 ·
メンテナーはふだん 1 日以内に返信
github/copilot-sdk の issue をすべて見る
似ている issue
-
area/dashboard kind/bug QA/dev-automation
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
rancher/dashboard#19379 · コメント 2 件 ·
メンテナーはふだん 5 日以内に返信
-
perf(core): getComments() runs the approved count and the comment list as two sequential queriesオープンarea/core bot:bug bot:working
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
emdash-cms/emdash#3905 · コメント 2 件 ·
メンテナーはふだん 1 日以内に返信
-
community first-timers-only good first issue hacktoberfest help wanted low hanging fruit up-for-grabs
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
lingdojo/kana-dojo#31728 · コメント 1 件 · リアクション 5 件 ·
メンテナーはふだん 1 日以内に返信
-
selective-claw: freshTailTurns=0 keeps ALL turns verbatim and summarizes none (slice(-0) === slice(0))対応中かも @zjncs が今日担当しました。 オープンcomponent:tokenless
難易度 2/5 1〜3時間 初心者へのやさしさ 80/100
agentic-os-org/ANOLISA#6112 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
bug needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
rjsf-team/react-jsonschema-form#5439 ·
メンテナーはふだん 1 日以内に返信