Skip to content

fix: Stop running tests twice on pull requests#31

Merged
jason-famedly merged 1 commit into
masterfrom
jason/stop-double-testing-ci
May 14, 2025
Merged

fix: Stop running tests twice on pull requests#31
jason-famedly merged 1 commit into
masterfrom
jason/stop-double-testing-ci

Conversation

@jason-famedly
Copy link
Copy Markdown
Member

No description provided.

@jason-famedly jason-famedly requested a review from a team as a code owner April 30, 2025 15:42
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 78.58%. Comparing base (3fe8d8c) to head (c082913).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #31      +/-   ##
==========================================
- Coverage   78.66%   78.58%   -0.08%     
==========================================
  Files         472      472              
  Lines       67655    67540     -115     
  Branches    10419    10415       -4     
==========================================
- Hits        53219    53076     -143     
- Misses      10918    10927       +9     
- Partials     3518     3537      +19     

see 63 files with indirect coverage changes


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 3fe8d8c...c082913. 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.

Comment thread .github/workflows/famedly-tests.yml Outdated
@jason-famedly jason-famedly force-pushed the jason/stop-double-testing-ci branch from f5c544d to f0eb75f Compare May 14, 2025 11:12
…oid double testing in CI when any change is made to a pull request
@jason-famedly jason-famedly force-pushed the jason/stop-double-testing-ci branch from f0eb75f to c082913 Compare May 14, 2025 11:21
@jason-famedly jason-famedly merged commit e685958 into master May 14, 2025
9 checks passed
@jason-famedly jason-famedly deleted the jason/stop-double-testing-ci branch August 8, 2025 10:12
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