Skip to content

CI: Remove approve-package-publishing job from packages publishing - #34783

Merged
r-farkhutdinov merged 3 commits into
DevExpress:mainfrom
r-farkhutdinov:protect-workflows-fix
Aug 17, 2026
Merged

CI: Remove approve-package-publishing job from packages publishing#34783
r-farkhutdinov merged 3 commits into
DevExpress:mainfrom
r-farkhutdinov:protect-workflows-fix

Conversation

@r-farkhutdinov

Copy link
Copy Markdown
Contributor

No description provided.

@r-farkhutdinov r-farkhutdinov self-assigned this Aug 17, 2026
Copilot AI lite review requested due to automatic review settings August 17, 2026 07:29
@r-farkhutdinov
r-farkhutdinov requested a review from a team as a code owner August 17, 2026 07:29

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the GitHub Actions package publishing workflow to remove the dedicated “approval” job and instead apply environment gating directly within the publishing stage.

Changes:

  • Removed the approve-package-publishing job and the build job dependency/condition that waited for it.
  • Added a conditional environment: assignment to the publish job to select between packages-publishing and packages-publishing-automation.
Suppressed comments (1)

.github/workflows/packages_publishing.yml:169

  • The environment expression will evaluate to the boolean true when inputs.tag == 'stable' (because || short-circuits before the && 'packages-publishing' part). GitHub Actions expects an environment name (string), so this can break stable publishing runs.
    needs: build

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment on lines 44 to 46
build:
name: Build packages
runs-on: ubuntu-latest
Copilot AI review requested due to automatic review settings August 17, 2026 07:31

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/packages_publishing.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Signed-off-by: Ruslan Farkhutdinov <23283554+r-farkhutdinov@users.noreply.github.com>
Copilot AI review requested due to automatic review settings August 17, 2026 08:14

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.

Comment thread .github/workflows/packages_publishing.yml
@r-farkhutdinov
r-farkhutdinov added this pull request to the merge queue Aug 17, 2026
@github-merge-queue
github-merge-queue Bot removed this pull request from the merge queue due to failed status checks Aug 17, 2026
@r-farkhutdinov
r-farkhutdinov added this pull request to the merge queue Aug 17, 2026
Merged via the queue into DevExpress:main with commit edebfdc Aug 17, 2026
125 of 126 checks passed
@r-farkhutdinov
r-farkhutdinov deleted the protect-workflows-fix branch August 17, 2026 10:09
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.

3 participants