Skip to content

Upstream sync: 3 new commits (2026-04-07) — docs-only, no Java SDK changes needed#59

Merged
brunoborges merged 2 commits intomainfrom
copilot/upstream-sync-3-new-commits
Apr 7, 2026
Merged

Upstream sync: 3 new commits (2026-04-07) — docs-only, no Java SDK changes needed#59
brunoborges merged 2 commits intomainfrom
copilot/upstream-sync-3-new-commits

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 7, 2026

Analyzed 3 upstream commits from github/copilot-sdk. All changes are documentation-only (Java language tabs added to the upstream multi-language docs site) plus one CI workflow change. The Java SDK already has comprehensive, correct Java-specific documentation covering all affected areas.

Upstream commits analyzed

Commit Description Action
200bfef Add workflow_dispatch to triage feedback workflow Not ported — CI-only change in upstream repo
156cf1f Add Java language tabs across all SDK documentation Not ported — Java SDK already has Java-specific docs
c3fa6cb Fix code review comments on Java doc snippets (missing imports, missing PermissionHandler.APPROVE_ALL) Not ported — Java SDK docs already use correct imports and patterns

Not ported (intentionally)

  • Java language tab additions (156cf1f, c3fa6cb) — These changes add/fix Java examples in the upstream's multi-language docs site. The Java SDK maintains its own Java-specific documentation in src/site/markdown/ which already covers all affected features with correct code examples and imports.
  • Triage workflow change (200bfef) — GitHub Actions workflow internal to the upstream repo; not applicable here.

Documentation Impact

No documentation changes required. The upstream's Java examples are equivalent to (or in some cases less accurate than) what the Java SDK docs already provide — e.g., the upstream fix added PermissionHandler.APPROVE_ALL and missing java.util.* imports that our docs already had correct.

Changes

  • Updated .lastmerge to c3fa6cbfb83d4a20b7912b1a17013d48f5a277a1

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)
  • mvn spotless:apply has been run to format the code
  • mvn clean verify passes locally

Does this introduce a breaking change?

  • Yes
  • No

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/graphql
    • Triggering command: /usr/bin/gh gh pr edit --add-label upstream-sync (http block)
    • Triggering command: /usr/bin/gh gh pr list --head copilot/upstream-sync-3-new-commits (http block)
    • Triggering command: /usr/bin/gh gh issue edit 59 --add-label upstream-sync (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Co-authored-by: edburns <75821+edburns@users.noreply.github.com>
Copilot AI changed the title [WIP] Sync upstream with 3 new commits from Copilot SDK Upstream sync: 3 new commits (2026-04-07) — docs-only, no Java SDK changes needed Apr 7, 2026
Copilot AI requested a review from edburns April 7, 2026 09:33
@brunoborges brunoborges marked this pull request as ready for review April 7, 2026 13:38
Copilot AI review requested due to automatic review settings April 7, 2026 13:38
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the repository’s recorded upstream sync point to reflect the latest analyzed upstream commit (docs/CI-only upstream changes; no Java SDK ports needed).

Changes:

  • Update .lastmerge to c3fa6cbfb83d4a20b7912b1a17013d48f5a277a1
Show a summary per file
File Description
.lastmerge Advances the stored upstream commit SHA used by sync/release automation.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@brunoborges brunoborges merged commit 9642f67 into main Apr 7, 2026
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[upstream-sync] Upstream sync: 3 new commits (2026-04-07)

4 participants