⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@fhanau
Copy link
Contributor

@fhanau fhanau commented Dec 31, 2025

  • Use remote_download_minimal to speed up release CI
  • Don't disable caching for wpt tests, just need to tell Bazel to fetch the
    outputs.
  • Disable building benchmarks when this was already supposed to happen, this is
    achieved by fixing ci-limit-storage so that test_tag_filters is adjusted too.
  • Remove ci-limit-storage from ci-linux/ci-linux-arm, it is a no-op there
  • Adjust build_tag_filters on ci-linux/ci-linux-arm so that benchmark and
    container tests are built as part of bazel build instead of later in bazel
    test.
  • Use ci-linux-common in lint CI to run with default set of build filters, that
    way we don't have to build container targets
  • Fix clang-tidy warning now that lint CI applies to benchmark targets

@fhanau fhanau force-pushed the felix/123125-overhaul-ci branch from 94afa6b to 36e60fb Compare January 26, 2026 00:09
@codecov-commenter
Copy link

codecov-commenter commented Jan 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 70.14%. Comparing base (66745cd) to head (9500b86).

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #5805      +/-   ##
==========================================
+ Coverage   70.01%   70.14%   +0.13%     
==========================================
  Files         406      407       +1     
  Lines      107106   107188      +82     
  Branches    17965    17965              
==========================================
+ Hits        74988    75192     +204     
+ Misses      21321    21209     -112     
+ Partials    10797    10787      -10     

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

@fhanau fhanau force-pushed the felix/123125-overhaul-ci branch from 36e60fb to 152cad8 Compare January 26, 2026 01:32
@fhanau fhanau marked this pull request as ready for review January 26, 2026 01:32
@fhanau fhanau requested review from a team as code owners January 26, 2026 01:32
@codspeed-hq
Copy link

codspeed-hq bot commented Jan 26, 2026

Merging this PR will not alter performance

✅ 70 untouched benchmarks
⏩ 129 skipped benchmarks1


Comparing felix/123125-overhaul-ci (9500b86) with main (66745cd)

Open in CodSpeed

Footnotes

  1. 129 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

- Use remote_download_minimal to speed up release CI
- Don't disable caching for wpt tests, just need to tell Bazel to fetch the
  outputs.
- Disable building benchmarks when this was already supposed to happen, this is
  achieved by fixing ci-limit-storage so that test_tag_filters is adjusted too.
- Remove ci-limit-storage from ci-linux/ci-linux-arm, it is a no-op there
- Adjust build_tag_filters on ci-linux/ci-linux-arm so that benchmark and
  container tests are built as part of bazel build instead of later in bazel
  test.
- Use ci-linux-common in lint CI to run with default set of build filters, that
  way we don't have to build container targets
- Fix clang-tidy warning now that lint CI applies to benchmark targets
@fhanau fhanau force-pushed the felix/123125-overhaul-ci branch from 152cad8 to 9500b86 Compare January 26, 2026 18:59
@fhanau fhanau merged commit a0746ee into main Jan 26, 2026
22 checks passed
@fhanau fhanau deleted the felix/123125-overhaul-ci branch January 26, 2026 19:20
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.

3 participants