Skip to content

AP-25847 Remove unused code from "Extension Bundling" pipeline#9

Open
HedgehogCode wants to merge 2 commits intomasterfrom
bug/AP-25847-remove-unreachable-step
Open

AP-25847 Remove unused code from "Extension Bundling" pipeline#9
HedgehogCode wants to merge 2 commits intomasterfrom
bug/AP-25847-remove-unreachable-step

Conversation

@HedgehogCode
Copy link
Copy Markdown
Contributor

No description provided.

… CI pipeline

AP-25847 (Python Extension Template: Bundling GitHub CI workflow cannot publish the artifacts)
AP-25847 (Python Extension Template: Bundling GitHub CI workflow cannot publish the artifacts)
Copy link
Copy Markdown

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

Removes unused manual-run configuration and artifact handling from the “Extension Bundling” GitHub Actions workflow to simplify the pipeline.

Changes:

  • Removed workflow_dispatch inputs (log level) from the workflow triggers.
  • Removed conditional artifact upload step intended only for manual runs.
  • Simplified the build summary output by removing manual-run-only fields/artifact section.

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

Comment on lines 3 to 7
on:
push:
branches: [ "main", "master" ]
pull_request:
branches: [ "main", "master" ]
Copy link

Copilot AI Apr 13, 2026

Choose a reason for hiding this comment

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

Since workflow_dispatch was removed from the on: triggers, the job-level condition if: ${{ github.event_name != 'workflow_dispatch' }} is now dead/always true and can be removed to avoid confusion (or reintroduce workflow_dispatch if manual runs are still intended).

Copilot uses AI. Check for mistakes.
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