Skip to content

Conversation

@matejchalk
Copy link
Collaborator

Fixes a couple of problems with Codecov job:

To make the workflow more robust and require less maintenance, I've added a script which generates the GitHub Actions matrix strategy from our Nx graph. Currently, that's:

project:
- ci
- cli
- core
- create-cli
- models
- nx-plugin
- plugin-coverage
- plugin-eslint
- plugin-js-packages
- plugin-jsdocs
- plugin-lighthouse
- plugin-typescript
- utils
target:
- unit-test
- int-test
exclude:
- project: create-cli
  target: int-test
- project: models
  target: int-test
- project: plugin-lighthouse
  target: int-test

I've tested the workflow via temporary trigger here.

@matejchalk matejchalk self-assigned this Aug 27, 2025
@github-actions github-actions bot added 🦾 CI/CD Continuous integration and deployment 🛠️ tooling labels Aug 27, 2025
@matejchalk matejchalk added the 🔬 testing writing tests label Aug 27, 2025
@nx-cloud
Copy link

nx-cloud bot commented Aug 27, 2025

View your CI Pipeline Execution ↗ for commit 21e025d

Command Status Duration Result
nx code-pushup -- compare --verbose ✅ Succeeded 7s View ↗
nx code-pushup -- --verbose ✅ Succeeded 5m 11s View ↗
nx code-pushup -- print-config --verbose --outp... ✅ Succeeded 6s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-27 15:31:00 UTC

@nx-cloud
Copy link

nx-cloud bot commented Aug 27, 2025

View your CI Pipeline Execution ↗ for commit 21e025d

Command Status Duration Result
nx affected -t e2e-test --parallel=1 ✅ Succeeded <1s View ↗
nx affected:lint --parallel=3 ✅ Succeeded <1s View ↗
nx affected -t unit-test --parallel=3 --coverag... ✅ Succeeded <1s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-27 15:24:47 UTC

@pkg-pr-new
Copy link

pkg-pr-new bot commented Aug 27, 2025

Open in StackBlitz

@code-pushup/ci

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/ci@1088

@code-pushup/cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/cli@1088

@code-pushup/core

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/core@1088

@code-pushup/create-cli

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/create-cli@1088

@code-pushup/models

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/models@1088

@code-pushup/nx-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/nx-plugin@1088

@code-pushup/coverage-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/coverage-plugin@1088

@code-pushup/eslint-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/eslint-plugin@1088

@code-pushup/js-packages-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/js-packages-plugin@1088

@code-pushup/jsdocs-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/jsdocs-plugin@1088

@code-pushup/lighthouse-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/lighthouse-plugin@1088

@code-pushup/typescript-plugin

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/typescript-plugin@1088

@code-pushup/utils

npm i https://pkg.pr.new/code-pushup/cli/@code-pushup/utils@1088

commit: 21e025d

@github-actions
Copy link
Contributor

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 168bfc3 with previous commit 163f67f.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Performance 🔴 32 🔴 33 ↑ +1
Code coverage 🟢 91 🟢 91
Security 🟡 64 🟡 64
Updates 🟡 85 🟡 85
Accessibility 🟢 92 🟢 92
Best Practices 🟢 100 🟢 100
SEO 🟡 61 🟡 61
Type Safety 🟢 100 🟢 100
Bug prevention 🟢 100 🟢 100
Miscellaneous 🟢 100 🟢 100
Code style 🟢 100 🟢 100
Documentation 🔴 23 🔴 23
👍 1 group improved, 👍 4 audits improved, 👎 1 audit regressed, 12 audits changed without impacting score

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Lighthouse Performance 🔴 32 🔴 33 ↑ +1

20 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Lighthouse Avoids enormous network payloads 🟨 Total size was 2,680 KiB 🟩 Total size was 2,666 KiB ↓ −0.5 %
Lighthouse Speed Index 🟥 7.6 s 🟥 7.1 s ↓ −6.1 %
Lighthouse First Contentful Paint 🟥 3.3 s 🟥 3.2 s ↓ −1.7 %
Lighthouse Time to Interactive 🟥 17.1 s 🟥 18.4 s ↑ +7.5 %
Lighthouse Total Blocking Time 🟥 3,890 ms 🟥 3,160 ms ↓ −18.9 %
Lighthouse Minimizes main-thread work 🟥 14.2 s 🟥 12.3 s ↓ −13.3 %
Lighthouse Server Backend Latencies 🟩 1,910 ms 🟩 300 ms ↓ −84.2 %
Lighthouse Metrics 🟩 100% 🟩 100% ↑ +7.5 %
Lighthouse JavaScript execution time 🟥 5.4 s 🟥 6.3 s ↑ +16 %
Lighthouse Max Potential First Input Delay 🟥 1,210 ms 🟥 1,930 ms ↑ +59.3 %
Lighthouse Uses efficient cache policy on static assets 🟨 30 resources found 🟨 30 resources found ↓ −0.1 %
Lighthouse Reduce unused JavaScript 🟥 Potential savings of 582 KiB 🟥 Potential savings of 582 KiB ↑ +31.3 %
Lighthouse Largest Contentful Paint 🟥 11.6 s 🟥 11.6 s ↑ +0.3 %
Lighthouse Initial server response time was short 🟩 Root document took 560 ms 🟩 Root document took 520 ms ↓ −6 %
Lighthouse Network Round Trip Times 🟩 40 ms 🟩 70 ms ↑ +81.5 %
Lighthouse Reduce unused CSS 🟥 Potential savings of 113 KiB 🟥 Potential savings of 113 KiB ↓ −6.2 %
Lighthouse Remove duplicate modules in JavaScript bundles 🟥 Potential savings of 100 KiB 🟥 Potential savings of 98 KiB ↓ −6.2 %

593 other audits are unchanged.

Copy link
Collaborator

@BioPhoton BioPhoton left a comment

Choose a reason for hiding this comment

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

Thx for the fix

@matejchalk matejchalk merged commit 5b4069a into main Aug 27, 2025
18 checks passed
@matejchalk matejchalk deleted the fix-codecov branch August 27, 2025 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🦾 CI/CD Continuous integration and deployment 🔬 testing writing tests 🛠️ tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants