Skip to content

Conversation

mishraomp
Copy link
Collaborator

No description provided.

Copilot

This comment was marked as outdated.

@mishraomp mishraomp added the bug Something isn't working label Jun 16, 2025
Copy link
Contributor

@Copilot 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

This PR updates the PROD release workflow to support auto-releasing based on GitHub release events.

  • Added a new release trigger for published releases.
  • Updated output variable fallbacks and guarded steps with conditions to differentiate between release events and manual triggers.
Comments suppressed due to low confidence (2)

.github/workflows/release.yml:31

  • Consider adding a comment to clarify why the Conventional Changelog Update step is skipped for release events. This can help future maintainers understand the intended workflow behavior when a GitHub release event occurs.
if: (github.event_name != 'release')

.github/workflows/release.yml:80

  • Review whether the Github Release job should be skipped on release events given that the workflow is now triggered by release events. Adding inline documentation on this condition would clarify the intended behavior.
if: (needs.vars.outputs.tag != '' && github.event_name != 'release')

@mishraomp mishraomp enabled auto-merge (squash) June 16, 2025 23:47
@mishraomp mishraomp disabled auto-merge June 17, 2025 01:59
@mishraomp mishraomp merged commit 7eeb27b into main Jun 17, 2025
19 checks passed
@mishraomp mishraomp deleted the fix/release-github branch June 17, 2025 01:59
@github-project-automation github-project-automation bot moved this from New to Done in NRIDS - Architecture Jun 17, 2025
@mishraomp mishraomp linked an issue Jun 17, 2025 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Deploy to PROD based on Github Release
1 participant