Skip to content

ci: skip pr build for draft pull requests#126

Merged
j-d-ha merged 1 commit into
mainfrom
fix/set-build-to-not-run-for-draft
Nov 11, 2025
Merged

ci: skip pr build for draft pull requests#126
j-d-ha merged 1 commit into
mainfrom
fix/set-build-to-not-run-for-draft

Conversation

@j-d-ha

@j-d-ha j-d-ha commented Nov 11, 2025

Copy link
Copy Markdown
Collaborator

Summary

  • Add condition to skip build-and-test and code-format jobs when a pull request is in draft status
  • Prevents unnecessary CI resource usage until the PR is ready for review

Changes

  • Updated .github/workflows/pr-build.yaml to check github.event.pull_request.draft == false for both jobs

Testing

Manual verification that the workflow condition is correctly applied to both jobs

Add condition to skip build and code-format jobs when PR is in draft
status. This prevents unnecessary CI resource usage until PR is ready
for review.
@github-actions github-actions Bot added the type: ci CI/CD changes label Nov 11, 2025
@sonarqubecloud

Copy link
Copy Markdown

@codecov

codecov Bot commented Nov 11, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #126   +/-   ##
=======================================
  Coverage   50.12%   50.12%           
=======================================
  Files          52       52           
  Lines        1640     1640           
  Branches      204      204           
=======================================
  Hits          822      822           
  Misses        765      765           
  Partials       53       53           

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 0d4ed7a...65fbd29. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@j-d-ha
j-d-ha merged commit ddf469c into main Nov 11, 2025
8 checks passed
@j-d-ha
j-d-ha deleted the fix/set-build-to-not-run-for-draft branch November 11, 2025 02:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: ci CI/CD changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant