Skip to content

Commit 5f0e1bd

Browse files
committed
Update progress
1 parent 6e369ac commit 5f0e1bd

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

80-java-monorepo-add-01-remove-before-merge/dd-2989727-move-java-to-monorepo-plan.md

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222

2323
- [] **Provision secrets** in `github/copilot-sdk` (see §2A) See https://github.com/github/copilot-sdk-partners/issues/90
2424
- [] **Verify CODEOWNERS team** access. See https://github.com/github/copilot-sdk-partners/issues/89
25-
- [] **Check Maven Central Trusted Publisher** — can `github/copilot-sdk` publish to `com.github:copilot-sdk-java`? See
25+
- [] **Check Maven Central Trusted Publisher** — can `github/copilot-sdk` publish to `com.github:copilot-sdk-java`? See
2626
- [] **Check GitHub Pages** — is it enabled? Can Java docs coexist? See https://github.com/github/copilot-sdk-partners/issues/85
2727
- [ ] **Confirm branch protection** — will new required status checks be accepted?
2828
- [ ] **Create tracking issue** in `github/copilot-sdk` for this migration
@@ -378,7 +378,7 @@ What changes is the **mechanism**: instead of polling a remote repository, the w
378378
| **Composite Action:** `test-report` | `.github/actions/test-report/` | Test report generation | Yes |
379379
| **Scripts:** `release/`, `ci/`, `build/`, `reference-impl-sync/` | `.github/scripts/` | Release, CI, sync automation | Yes — **path rewrites** |
380380
| **Dependabot:** `dependabot.yml` | `.github/` | Maven + GitHub Actions updates | Merge into monorepo's `dependabot.yaml` |
381-
| **CODEOWNERS** | `.github/` | `@github/copilot-sdk-java` | Merge into monorepo's CODEOWNERS |
381+
| **CODEOWNERS** | `.github/` | ~~`@github/copilot-sdk-java` | Merge into monorepo's CODEOWNERS~~ |
382382
| **Issue Templates:** bug, documentation, feature, maintenance | `.github/ISSUE_TEMPLATE/` | Issue forms | Assess whether monorepo issue triage covers this |
383383
| **PR Template** | `.github/pull_request_template.md` | PR form | Merge or keep per-language |
384384
| **Release Config** | `.github/release.yml` | Auto-generated release notes config | Merge |
@@ -424,7 +424,7 @@ What changes is the **mechanism**: instead of polling a remote repository, the w
424424
| M1 | **Codegen `package.json` merge** | Java codegen has its own `@github/copilot` dependency; monorepo codegen gets it from `nodejs/node_modules` | Align Java codegen to use the same dependency source. May need to add `generate:java` script to monorepo's `scripts/codegen/package.json`. |
425425
| M2 | **GitHub Pages conflict** | Java deploys versioned docs to Pages. Monorepo may have its own Pages setup. | Use subdirectory deployment or a separate Pages branch for Java. |
426426
| M3 | **Branch protection / required checks** | New `java-sdk-tests` check may not be in the required list | Add to branch protection after first successful run. |
427-
| M4 | **CODEOWNERS team permissions** | `@github/copilot-sdk-java` team may not have write access to `github/copilot-sdk` | Verify team access and add to repo collaborators. |
427+
| M4 | **CODEOWNERS team permissions** | `@github/copilot-sdk-java` team may not have write access to `github/copilot-sdk` | Verify team access and add to repo collaborators. See https://github.com/github/copilot-sdk-partners/issues/89 |
428428
| M5 | **`copilot-setup-steps.yml` bloat** | Adding JDK + Maven makes agent setup slower for non-Java tasks | Acceptable trade-off; other languages already add their tools. Could consider conditional setup but that's over-engineering. |
429429
| M6 | **gh-aw version mismatch** | Java repo uses gh-aw `v0.68.3` setup action pinned at `v0.71.5`; monorepo uses `v0.64.2` reference in docs | Align gh-aw versions. Use the newer version. Recompile all `.lock.yml` files. |
430430

@@ -455,7 +455,7 @@ What changes is the **mechanism**: instead of polling a remote repository, the w
455455

456456
- [ ] `copilot-setup-steps.yml` includes JDK and Maven
457457
- [ ] `dependabot.yaml` includes Maven ecosystem for `java/`
458-
- [ ] `CODEOWNERS` includes `java/` path
458+
- [] `CODEOWNERS` includes `java/` path. See https://github.com/github/copilot-sdk-partners/issues/89
459459
- [ ] `justfile` has all Java targets and `just test` includes Java
460460
- [ ] `sdk-consistency-review` includes `java/` in path triggers
461461
- [ ] `issue-triage` knows about `sdk/java` label

0 commit comments

Comments
 (0)