Skip to content

Update breakage: use organization-wide workflow#157

Open
MaxenceGollier wants to merge 1 commit into
JuliaSmoothOptimizers:mainfrom
MaxenceGollier:update-breakage
Open

Update breakage: use organization-wide workflow#157
MaxenceGollier wants to merge 1 commit into
JuliaSmoothOptimizers:mainfrom
MaxenceGollier:update-breakage

Conversation

@MaxenceGollier
Copy link
Copy Markdown
Contributor

Copilot AI review requested due to automatic review settings June 1, 2026 20:29
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

Note

Copilot was unable to run its full agentic suite in this review.

Refactors the Breakage workflow to use the centralized reusable workflows from the JuliaSmoothOptimizers/.github repository, removing local implementations.

Changes:

  • Replaces inline Breakage workflow with a call to the reusable workflow from JuliaSmoothOptimizers/.github.
  • Adds a new CommentBreakage workflow that runs on Breakage workflow completion and delegates to the reusable comment workflow.
  • Removes the now-unused .breakage/get_jso_users.jl script and its Project.toml.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
.github/workflows/Breakage.yml Replaced custom implementation with a call to the shared reusable workflow.
.github/workflows/CommentBreakage.yml New workflow triggered on Breakage completion to post PR comments via reusable workflow.
.breakage/get_jso_users.jl Removed; functionality moved to the centralized workflow.
.breakage/Project.toml Removed; no longer needed after script deletion.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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