Skip to content

[SDPTA-1431] Improve log for set status#72

Merged
tim-yao merged 4 commits intomainfrom
sdpta-1431-set-status
Sep 9, 2025
Merged

[SDPTA-1431] Improve log for set status#72
tim-yao merged 4 commits intomainfrom
sdpta-1431-set-status

Conversation

@tim-yao
Copy link
Contributor

@tim-yao tim-yao commented Sep 8, 2025

@tim-yao tim-yao self-assigned this Sep 8, 2025
klaukl
klaukl previously approved these changes Sep 8, 2025
@klaukl
Copy link
Contributor

klaukl commented Sep 8, 2025

Do we want to run the latest upgrade on reference/content-ref to check e2e upgrade all good?

@tim-yao tim-yao requested a review from Copilot September 9, 2025 00:21
Copy link

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 improves logging and documentation for status-related GitHub Actions workflows. The changes enhance visibility into workflow status operations and add a new action for status calculation.

  • Enhanced the description of the state input parameter to include valid values
  • Added logging to display the current workflow status
  • Introduced a new composite action to calculate and return workflow status based on job results

Reviewed Changes

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

File Description
.github/workflows/set_status.yml Enhanced input description and added status logging
.github/actions/test-get-status.yml New composite action for calculating workflow status from job results

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@tim-yao tim-yao merged commit 1ca8d14 into main Sep 9, 2025
@tim-yao tim-yao deleted the sdpta-1431-set-status branch September 9, 2025 00:25
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