Skip to content

Include Java in copilot dependency codegen#1533

Closed
stephentoub wants to merge 1 commit into
mainfrom
stephentoub/add-java-generator
Closed

Include Java in copilot dependency codegen#1533
stephentoub wants to merge 1 commit into
mainfrom
stephentoub/add-java-generator

Conversation

@stephentoub
Copy link
Copy Markdown
Collaborator

The automated @github/copilot dependency update workflow refreshes generated SDK code after pulling in a new runtime package. Java keeps its generator in java/scripts/codegen, so it was not being updated or run by the shared workflow.

Changes

  • Update @github/copilot in java/scripts/codegen during the dependency update.
  • Run the Java code generator after the shared codegen step.
  • Keep the generated commit and PR text language-neutral now that all generators run.

Generated by Copilot

Run Java codegen as part of the @github/copilot dependency update workflow.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Copilot AI review requested due to automatic review settings June 1, 2026 16:38
@stephentoub stephentoub requested a review from a team as a code owner June 1, 2026 16: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 automated @github/copilot dependency bump workflow so it also updates and runs the Java generator, ensuring Java’s generated sources stay in sync when the runtime package changes.

Changes:

  • Update @github/copilot in java/scripts/codegen as part of the dependency bump workflow.
  • Run Java codegen (java/scripts/codegen) after the shared codegen step.
  • Make the generated commit message and PR body language-neutral now that all generators run.
Show a summary per file
File Description
.github/workflows/update-copilot-dependency.yml Extends the dependency update workflow to include Java codegen dependency update + generation, and adjusts commit/PR text accordingly.

Copilot's findings

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

@stephentoub stephentoub closed this Jun 1, 2026
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.

2 participants