Repository navigation
ci: 文書JAR修正と最低Javaの検証をGitHub Actionsへ移す - #27
Conversation
lambig
left a comment
There was a problem hiding this comment.
再レビュー(Assistant Tech Lead/PM/PO)— CI整備を追加要求
判定:文書JARの修正は妥当。W06の受け入れに必要なCIが未整備/未確認のため、CI対応を要求します。
技術
javadocJar の入力をclass出力から javadoc.destinationDir に、依存を classes から javadoc に変更しており、誤ったclass格納に直接対応しています。 #471/#567に記録されたローカル結果は 21試験成功・HTML 15件・class 0件(head 1725ae0)で、今回の修正目的に対応しています。公開API・依存版の変更はありません。
CIの問題: 現行 .travis.yml は openjdk8 のみを指定する一方、build.gradle は sourceCompatibility = 1.11 です。さらに install: gradle assemble がシステムGradleを呼び、必要なテスト・文書JAR検査を明示していません。CIのJDK選定自体が不整合です。 当該headに紐づくGitHub Actions実行結果も取得できませんでした(Travisの外部実行状況は未確認)。
CI整備の要求(#567 W06)
- PRと既定ブランチの更新でCIを起動し、リポジトリの固定Wrapper
./gradlewでtest・javadocJar・sourcesJarを実行する。Travis修正またはGitHub Actions導入のどちらでも構いません。 - このライブラリの最低対応 Java 11 で実際にビルド/テストし、Wrapper Gradle 7.4.2 と実行JDKの組み合わせを揃える。旧設定のJava 8ジョブを、サポート対象と区別して修正すること。
- CIで生成された
*-javadoc.jarの内容を検査し、HTMLが1件以上・.classが0件をassertする。空JAR/class誤格納ならCIを失敗させる。 - 成功したCIの実行URLと対象head SHAをPRへ紐づける。
PM/PO
Javadoc型パラメータ警告4件は残件として#567に記録してください。公開APIのTuple2依存をconsumerで解決できるかはG2/G3で別に検証してください。 このレビューは対象のJavadoc修正の正しさを認める一方、ローカル検証をCI成功・W06完了と読み替えないためのものです。CI整備を別PRに分ける場合、そのPRと完了証拠へ相互リンクし、W06側のCI整備を未完了として管理してください。公開・署名・Centralからの取得は後続のG4/G5受け入れとして扱えます。
|
CI整備の指摘に対応し、同じPRにGitHub Actionsを追加しました。旧Travis設定を削除し、PR・既定ブランチ更新で固定Wrapperを使用します。 対象head: Java 11でclean・test・javadocJar・sourcesJarと内容検査が成功しました。CI生成物はJavadocのHTML 15件、sourcesのJavaファイル 3件で、両JARともclass 0件です。試験結果と両JARも同じ実行のartifactに保存されています。 空/欠落/複数JAR/class混入で失敗する検査を追加し、ローカルで正常・異常7条件を確認しました。actionlintも成功しています。既定ブランチ更新のトリガーは設定済みで、実行確認はマージ後です。文書本文・依存metadata・consumer・Central公開などは後続のままです。レビューの再確認をお願いします。 |
lambig
left a comment
There was a problem hiding this comment.
前回のCI整備指摘への対応確認(Tech Lead/PM/PO)
判定:指摘対応済み。このPRのJavadoc JAR修正+最低Java CI整備についてマージ可能。
Tech Lead: Javadoc JARのclass誤格納修正と、旧TravisのJava 8設定を最低対応Java 11へ合わせた変更を確認しました。既存Javadoc警告4件とTuple2公開依存のconsumer確認は#567/#158で追跡する対象です。 最新head e0df7bbabdbe539804eefb2991a386a09a8540dc について、PR起動CI の完了状態が success であること、Wrapperを使った clean test javadocJar sourcesJar(必要対象のJaCoCoを含む)、文書JAR検査、artifact保存の各ステップもすべて success であることを確認しました。検査スクリプトはJavadoc HTML/sources Javaの存在、class 0件、対象JARの一意性を検証しており、空JARや誤格納で失敗する実装です。
PM: CI結果はPRのhead SHAに対応し、成果物artifactも実際に存在します。したがってこの対象の文書生成・最低Java 11でのCI整備をW06の部分成果として受け入れて構いません。既定ブランチpushでの動作はマージ後にCIで確認してください。W06全体の完了は、他の対象および残る公開基盤・互換性の条件との照合で判断してください。
PO: 今回の配布文書の品質不良をPRの段階で検出できる状態に改善されています。独立consumerおよび公開版での取得・利用検証は後続ゲートで判定する方針と整合しています。前回レビューのCI関連指摘は解消とします。
Javadoc JARのclass誤格納を修正し、同じPRで旧TravisをGitHub Actionsへ移行します。ABService v1.11の既存ライブラリ整備(lambig/ABService#567 W06)のうち、文書成果物と最低対応JavaでのCIを扱います。
PRと既定ブランチ更新で、Java 11と固定Gradle Wrapperを使い、clean・test・javadocJar・sourcesJarを実行します。JavadocにHTML、sourcesにJavaソースが存在し、どちらもclassを含まないことを検査します。空・欠落・複数JAR・class混入はジョブ失敗になります。試験結果と文書JARをActionsのartifactへ保存します。
検証対象head:
e0df7bbabdbe539804eefb2991a386a09a8540dcGitHub Actions実行
ローカルではactionlintによる全workflowの構文検査、実際の文書JAR検査、正常・異常7条件の検査を確認しました。上記headのPR起動CIで、最低Javaの試験・文書生成・内容検査・artifact保存がすべて成功しました。既定ブランチへのpush実行はマージ後に確認します。
レビュー観点は、javadocタスクへの依存と生成先の接続、最低Javaでの実行、内容不備をCIが検出できることです。公開API・依存版は今回変更していません。Javadocの型パラメーター警告4件は残件です。toolchain更新・文書本文/依存metadataの整備、Java 25 consumer、署名・Central公開と公開版取得は後続で、W06全体の完了にはしていません。