Skip to content

chore: drop checkout repo from cloudrun deploy job#51

Merged
smelchior merged 1 commit intov1from
chore/improve-cloudrun
Feb 18, 2026
Merged

chore: drop checkout repo from cloudrun deploy job#51
smelchior merged 1 commit intov1from
chore/improve-cloudrun

Conversation

@smelchior
Copy link
Contributor

The checkout is not needed here as the deploy is not based on the repo

@smelchior smelchior requested a review from marek-saji February 18, 2026 16:15
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 18, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Summary by CodeRabbit

  • Chores
    • Simplified deployment workflow configuration.

Walkthrough

Removed the actions/checkout@v6.0.2 step from the deploy_cloudrun job in the GitHub Actions workflow. The deploy_cloudrun job now proceeds without an explicit repository checkout action.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow
.github/workflows/deploy-cloudrun.yaml
Removed the actions/checkout@v6.0.2 step from the deploy_cloudrun job.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

  • feat: add cloudrun deploy job #29: Directly modifies the same .github/workflows/deploy-cloudrun.yaml workflow's deploy_cloudrun job and its checkout step configuration (addition vs. removal).

Suggested reviewers

  • marek-saji

Poem

🐰 A step removed with care so fine,
No checkout needed in the deploy line,
The workflow hops along with grace,
Leaner, faster—a simpler place! ✨

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly describes the main change: removing the checkout step from the cloudrun deploy job, which aligns with the changeset that removes the actions/checkout@v6.0.2 step.
Description check ✅ Passed The description directly explains the rationale for the change: the checkout is unnecessary because the deploy process doesn't require repository files, which is relevant to the changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch chore/improve-cloudrun

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@marek-saji marek-saji left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@smelchior smelchior merged commit 130b3e7 into v1 Feb 18, 2026
2 checks passed
@smelchior smelchior deleted the chore/improve-cloudrun branch February 18, 2026 16:28
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