Skip to content

Conversation

@angelayu0530
Copy link
Collaborator

@angelayu0530 angelayu0530 commented Jan 23, 2026

Description of changes

  • added target Url
  • Added pending status block to be able to see the deployment as when it triggers

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Screen.Recording.2026-01-23.at.9.50.43.PM.mov

@angelayu0530
Copy link
Collaborator Author

/deploy

@github-actions
Copy link
Contributor

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions
Copy link
Contributor

Title

647: git push --set-upstream origin add-link


PR Type

Enhancement


Description

  • Add clickable status link to staging deploy

Diagram Walkthrough

flowchart LR
  GHAction["GitHub Actions deploy-stg workflow"] -- "creates commit status" --> CommitStatus["Commit status on SHA"]
  CommitStatus -- "adds target_url=runUrl" --> RunPage["Actions run details page"]
Loading

File Walkthrough

Relevant files
Enhancement
deploy-stg.yml
Add target_url to commit status for staging deploy             

.github/workflows/deploy-stg.yml

  • Add target_url: runUrl to commit status payload
  • Make staging deployment status clickable to run details
+1/-0     

@github-actions
Copy link
Contributor

The command to deploy to staging for the commit 9365c13 has been triggered. View action run

@github-actions
Copy link
Contributor

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Context String

The commit status 'context' is hard-coded to "Successful deployment to staging". If this workflow can set failure states or is reused, consider making the context reflect the actual job name or environment dynamically to align with existing patterns and avoid confusion in status checks.

context: "Successful deployment to staging",
description: status ? undefined : "Successful deployment to staging is required",

@angelayu0530
Copy link
Collaborator Author

/deploy

@github-actions
Copy link
Contributor

Staging deployment was cancelled for commit 9365c13

View run

@github-actions
Copy link
Contributor

The command to deploy to staging for the commit 6544b72 has been triggered. View action run

@codecov
Copy link

codecov bot commented Jan 23, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Contributor

Staging deployment succeeded for commit 6544b72

View run

@angelayu0530 angelayu0530 changed the title 647: git push --set-upstream origin add-link 647: wip Jan 23, 2026
@github-actions
Copy link
Contributor

The command to deploy to staging for the commit 6544b72 has been triggered. View action run

@angelayu0530 angelayu0530 changed the title 647: wip 647: Make deploy to stg status clickable to view latest action Jan 24, 2026
@github-actions
Copy link
Contributor

Staging deployment succeeded for commit 6544b72

View run

@github-actions
Copy link
Contributor

The command to deploy to staging for the commit e8521a3 has been triggered. View action run

@github-actions
Copy link
Contributor

Staging deployment succeeded for commit e8521a3

View run

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