Skip to content

chore(deployments): fix actions/checkout typo - #6255

Merged
jmelahman merged 1 commit into
mainfrom
jamison/action-typo
Nov 14, 2025
Merged

chore(deployments): fix actions/checkout typo#6255
jmelahman merged 1 commit into
mainfrom
jamison/action-typo

Conversation

@jmelahman

@jmelahman jmelahman commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Description

Fixes a typo introduced in 530d6d8

How Has This Been Tested?

https://github.com/onyx-dot-app/onyx/actions/runs/19378089924

Additional Options

  • Override Linear Check

Summary by cubic

Fixes a typo in the pinned actions/checkout reference across the deployment workflow. Restores the checkout step and unblocks CI builds and deployments.

Written for commit 7f2337a. Summary will update automatically on new commits.

@jmelahman
jmelahman requested a review from a team as a code owner November 14, 2025 21:32
@greptile-apps

greptile-apps Bot commented Nov 14, 2025

Copy link
Copy Markdown
Contributor

Greptile Overview

Greptile Summary

Corrects a typo in the Git SHA hash for actions/checkout that was introduced in the previous deployment pipeline refactoring. The hash was missing a 'b' character (0eb9ba0beb9ba), which would cause workflow failures when GitHub Actions attempts to resolve the action reference. The fix is applied consistently across all four build jobs (web, web-cloud, backend, model-server).

Confidence Score: 5/5

  • This PR is safe to merge with zero risk
  • The change is a straightforward typo correction in a SHA hash reference that fixes a critical issue preventing workflows from running. The fix is consistent across all affected locations, has been tested successfully per the PR description, and introduces no logical changes to the workflow behavior.
  • No files require special attention

Important Files Changed

File Analysis

Filename Score Overview
.github/workflows/deployment.yml 5/5 Fixed typo in actions/checkout SHA hash across all four build jobs (was missing 'b' character)

Sequence Diagram

sequenceDiagram
    participant Dev as Developer
    participant GH as GitHub Actions
    participant Workflow as deployment.yml
    participant Checkout as actions/checkout

    Dev->>GH: Push tag or trigger workflow_dispatch
    GH->>Workflow: Trigger deployment workflow
    Workflow->>Workflow: Determine which components to build
    
    alt Build Web
        Workflow->>Checkout: Use actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955
        Checkout->>Workflow: Code checked out
        Workflow->>Workflow: Build and push web Docker image
    end
    
    alt Build Web Cloud
        Workflow->>Checkout: Use actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955
        Checkout->>Workflow: Code checked out
        Workflow->>Workflow: Build and push web-cloud Docker image
    end
    
    alt Build Backend
        Workflow->>Checkout: Use actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955
        Checkout->>Workflow: Code checked out
        Workflow->>Workflow: Build and push backend Docker image
    end
    
    alt Build Model Server
        Workflow->>Checkout: Use actions/checkout@08eba0b27e820071cde6df949e0beb9ba4906955
        Checkout->>Workflow: Code checked out
        Workflow->>Workflow: Build and push model-server Docker image
    end
Loading

@greptile-apps greptile-apps Bot 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.

1 file reviewed, no comments

Edit Code Review Agent Settings | Greptile

@jmelahman
jmelahman added this pull request to the merge queue Nov 14, 2025

@cubic-dev-ai cubic-dev-ai Bot 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.

No issues found across 1 file

Merged via the queue into main with commit 259bc9d Nov 14, 2025
70 of 72 checks passed
@jmelahman
jmelahman deleted the jamison/action-typo branch November 14, 2025 22: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.

2 participants