Skip to content

[GHA] Don't run front/back-end job on purely back/front-end pr#4035

Merged
imnasnainaec merged 2 commits intomasterfrom
gha-ignore-front-back
Nov 24, 2025
Merged

[GHA] Don't run front/back-end job on purely back/front-end pr#4035
imnasnainaec merged 2 commits intomasterfrom
gha-ignore-front-back

Conversation

@imnasnainaec
Copy link
Collaborator

@imnasnainaec imnasnainaec commented Nov 20, 2025

The backend docker build and frontend test coverage checks are two of our longest-running, so it's nice to skip them when they're completely irrelevant to the pr at hand.

Only adding the folders/files that are most frequently changed in isolation and that will always be irrelevant to the workflow. (Trying to come up with a comprehensive inclusion list instead would create a maintenance nightmare.)

This change is Reviewable

@imnasnainaec imnasnainaec self-assigned this Nov 20, 2025
@imnasnainaec imnasnainaec added the 🟩Low Low-priority PR label Nov 20, 2025
@github-actions github-actions bot added the github_actions Pull requests that update GitHub Actions code label Nov 20, 2025
@codecov
Copy link

codecov bot commented Nov 20, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.64%. Comparing base (9734714) to head (e6c0426).
⚠️ Report is 26 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4035   +/-   ##
=======================================
  Coverage   74.64%   74.64%           
=======================================
  Files         293      293           
  Lines       10829    10829           
  Branches     1355     1355           
=======================================
  Hits         8083     8083           
  Misses       2351     2351           
  Partials      395      395           
Flag Coverage Δ
backend 85.55% <ø> (ø)
frontend 66.23% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.

Copy link
Contributor

@jasonleenaylor jasonleenaylor left a comment

Choose a reason for hiding this comment

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

:lgtm:

@jasonleenaylor reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @imnasnainaec)

@imnasnainaec imnasnainaec merged commit 963d0de into master Nov 24, 2025
19 checks passed
@imnasnainaec imnasnainaec deleted the gha-ignore-front-back branch November 24, 2025 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code 🟩Low Low-priority PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants