Skip to content

Fix broken build - #41

Merged
PramodKumarYadav merged 3 commits into
mainfrom
fix-broken-build
Jun 7, 2026
Merged

Fix broken build#41
PramodKumarYadav merged 3 commits into
mainfrom
fix-broken-build

Conversation

@PramodKumarYadav

Copy link
Copy Markdown
Collaborator

No description provided.

Copilot AI review requested due to automatic review settings June 7, 2026 20:30

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 manual deploy GitHub Actions workflows to fix a broken dispatch/deploy path by adjusting workflow_dispatch inputs, moving input validation into the environment-specific workflows, and aligning deploy behavior with the image-tag dropdown produced by build-image.yml.

Changes:

  • Reworks workflow_dispatch inputs for staging/production deploys (moves sha below image_tag, updates description) and adds a validate-inputs job.
  • Removes the reusable workflow’s input validation step and updates its input docs/comments to reflect the “blank or sentinel” source-build behavior.
  • Reorders the generated image_tag options JSON in build-image.yml so “latest” / “(build from source)” come first.

Reviewed changes

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

File Description
.github/workflows/deploy-staging.yml Updates dispatch inputs and adds a pre-job validation gate before resolving SHA and calling _deploy.yml.
.github/workflows/deploy-production.yml Same as staging, plus keeps production release creation flow.
.github/workflows/build-image.yml Changes ordering of generated image_tag dropdown options JSON.
.github/workflows/_deploy.yml Updates input docs/comments and removes the prior validation step from the reusable deploy workflow.

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

Comment thread .github/workflows/deploy-staging.yml
Comment thread .github/workflows/deploy-production.yml

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 4 out of 4 changed files in this pull request and generated 4 comments.

Comment thread .github/workflows/deploy-staging.yml
Comment thread .github/workflows/deploy-production.yml
Comment thread .github/workflows/deploy-staging.yml
Comment thread .github/workflows/deploy-production.yml
@PramodKumarYadav
PramodKumarYadav merged commit c280978 into main Jun 7, 2026
2 checks passed
@PramodKumarYadav
PramodKumarYadav deleted the fix-broken-build branch June 7, 2026 21:10
@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

ci deploy success for c280978

Environment: ci | View app

@github-actions github-actions Bot added the deployed: ci Deployed to ci label Jun 7, 2026
@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

staging deploy failure for c280978

Environment: staging | View app

@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

staging deploy success for c280978

Environment: staging | View app

@github-actions github-actions Bot added the deployed: staging Deployed to staging label Jun 7, 2026
@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

production deploy success for c280978

Environment: production | View app

@github-actions github-actions Bot added the deployed: production Deployed to production label Jun 7, 2026
@github-actions

github-actions Bot commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

production deploy success for c280978

Environment: production | View app

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deployed: ci Deployed to ci deployed: production Deployed to production deployed: staging Deployed to staging

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants