Skip to content

feat(ci): add helper function for parsing configPatterns from json string #1079

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

matejchalk
Copy link
Collaborator

Exporting some utilities from @code-pushup/ci which will be useful for parsing new configPatterns (#1046) and searchCommits (#1056) options in GitHub Action and GitLab pipelines template.

Copy link

nx-cloud bot commented Aug 22, 2025

View your CI Pipeline Execution ↗ for commit 7e906d6

Command Status Duration Result
nx affected -t e2e-test --parallel=1 ✅ Succeeded 12m 31s View ↗
nx affected -t int-test --parallel=3 --coverage... ✅ Succeeded 1m 35s View ↗
nx affected -t unit-test --parallel=3 --coverag... ✅ Succeeded 51s View ↗
nx affected:lint --parallel=3 ✅ Succeeded 1m 28s View ↗
nx run-many --target=build --parallel=3 ✅ Succeeded 34s View ↗

☁️ Nx Cloud last updated this comment at 2025-08-22 17:21:29 UTC

Copy link

pkg-pr-new bot commented Aug 22, 2025

Open in StackBlitz

@code-pushup/create-cli

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

@code-pushup/models

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

@code-pushup/cli

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

@code-pushup/nx-plugin

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

@code-pushup/ci

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

@code-pushup/core

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

@code-pushup/coverage-plugin

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

@code-pushup/eslint-plugin

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

@code-pushup/js-packages-plugin

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

@code-pushup/lighthouse-plugin

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

@code-pushup/jsdocs-plugin

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

@code-pushup/typescript-plugin

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

@code-pushup/utils

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

commit: 7e906d6

Copy link

github-actions bot commented Aug 22, 2025

Code PushUp

🤨 Code PushUp report has both improvements and regressions – compared current commit 1edac0e with previous commit 97c31f8.

🕵️ See full comparison in Code PushUp portal 🔍

🏷️ Categories

🏷️ Category ⭐ Previous score ⭐ Current score 🔄 Score change
Performance 🔴 34 🔴 31 ↓ −3
Code coverage 🟢 91 🟢 91 ↑ +0.1
Documentation 🔴 23 🔴 23 ↓ −0.1
Security 🔴 0 🔴 0
Updates 🟡 77 🟡 77
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
👍 1 group improved, 👎 2 groups regressed, 👍 3 audits improved, 👎 6 audits regressed, 14 audits changed without impacting score

🗃️ Groups

🔌 Plugin 🗃️ Group ⭐ Previous score ⭐ Current score 🔄 Score change
Lighthouse Performance 🔴 34 🔴 31 ↓ −3
Code coverage Code coverage metrics 🟢 91 🟢 91 ↑ +0.1
JSDoc coverage Documentation coverage 🔴 23 🔴 23 ↓ −0.1

18 other groups are unchanged.

🛡️ Audits

🔌 Plugin 🛡️ Audit 📏 Previous value 📏 Current value 🔄 Value change
Lighthouse Speed Index 🟥 6.9 s 🟥 7.7 s ↑ +11 %
Lighthouse First Contentful Paint 🟥 3.1 s 🟥 3.3 s ↑ +7.7 %
Lighthouse Total Blocking Time 🟥 2,360 ms 🟥 6,340 ms ↑ +168.4 %
Lighthouse Time to Interactive 🟥 17.1 s 🟥 18.2 s ↑ +6.7 %
JSDoc coverage Functions coverage 🟥 516 undocumented functions 🟥 517 undocumented functions ↑ +0.2 %
JSDoc coverage Variables coverage 🟥 225 undocumented variables 🟥 227 undocumented variables ↑ +0.9 %
Code coverage Branch coverage 🟨 86.3 % 🟨 86.3 % ↑ +0.1 %
Code coverage Function coverage 🟩 93.2 % 🟩 93.2 % ↑ +0.1 %
Code coverage Line coverage 🟨 87.6 % 🟨 87.6 % ↑ +0.1 %
Lighthouse Avoids enormous network payloads 🟩 Total size was 2,656 KiB 🟩 Total size was 2,646 KiB ↓ −0.4 %
Lighthouse Minimizes main-thread work 🟥 11.8 s 🟥 20.0 s ↑ +69.2 %
Lighthouse JavaScript execution time 🟥 5.1 s 🟥 8.8 s ↑ +71.3 %
Lighthouse Metrics 🟩 100% 🟩 100% ↑ +6.7 %
Lighthouse Largest Contentful Paint 🟥 11.5 s 🟥 12.6 s ↑ +8.9 %
Lighthouse Max Potential First Input Delay 🟥 1,340 ms 🟥 2,150 ms ↑ +60.8 %
Lighthouse Reduce unused CSS 🟥 Potential savings of 116 KiB 🟥 Potential savings of 116 KiB ↓ −21.8 %
Lighthouse Remove duplicate modules in JavaScript bundles 🟥 Potential savings of 95 KiB 🟥 Potential savings of 98 KiB ↓ −34 %
Lighthouse Initial server response time was short 🟩 Root document took 590 ms 🟩 Root document took 440 ms ↓ −25.8 %
Lighthouse Uses efficient cache policy on static assets 🟨 30 resources found 🟨 30 resources found ↑ +0.1 %
Lighthouse Server Backend Latencies 🟩 540 ms 🟩 500 ms ↓ −7.8 %
Lighthouse Reduce unused JavaScript 🟥 Potential savings of 591 KiB 🟥 Potential savings of 614 KiB ↓ −1.3 %
Lighthouse Network Round Trip Times 🟩 70 ms 🟩 60 ms ↓ −9.5 %
Lighthouse Cumulative Layout Shift 🟩 0.001 🟩 0 ↓ −100 %

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

Thanks for adding this helper. Added a suggestion for a const for export.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants