-
Notifications
You must be signed in to change notification settings - Fork 15
feat(ci): style top-level logs #1156
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
Conversation
|
View your CI Pipeline Execution ↗ for commit 76b0616
☁️ Nx Cloud last updated this comment at |
@code-pushup/ci
@code-pushup/cli
@code-pushup/core
@code-pushup/create-cli
@code-pushup/models
@code-pushup/nx-plugin
@code-pushup/axe-plugin
@code-pushup/coverage-plugin
@code-pushup/eslint-plugin
@code-pushup/js-packages-plugin
@code-pushup/jsdocs-plugin
@code-pushup/lighthouse-plugin
@code-pushup/typescript-plugin
@code-pushup/utils
@code-pushup/models-transformers
commit: |
Code PushUp🤨 Code PushUp report has both improvements and regressions – compared current commit 690a128 with previous commit 513f187. 🕵️ See full comparison in Code PushUp portal 🔍 🏷️ Categories👍 2 groups improved, 👎 1 group regressed, 👍 8 audits improved, 👎 3 audits regressed, 11 audits changed without impacting score🗃️ Groups
20 other groups are unchanged. 🛡️ Audits
656 other audits are unchanged. |
BioPhoton
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Left a question
5973d5a to
3cc1629
Compare
3cc1629 to
562fc88
Compare
Code PushUp🤨 Code PushUp report has both improvements and regressions – compared current commit 690a128 with previous commit 513f187. 💼 Project
|
| 🏷️ Category | ⭐ Previous score | ⭐ Current score | 🔄 Score change |
|---|---|---|---|
| Documentation | 🟡 67 | 🟡 68 | |
| Code coverage | 🟢 92 | 🟢 92 |
4 other categories are unchanged.
👍 1 group improved, 👎 1 group regressed, 👍 3 audits improved, 👎 1 audit regressed, 1 audit changed without impacting score
🗃️ Groups
| 🔌 Plugin | 🗃️ Group | ⭐ Previous score | ⭐ Current score | 🔄 Score change |
|---|---|---|---|---|
| JSDoc coverage | Documentation coverage | 🟡 67 | 🟡 68 | |
| Code coverage | Code coverage metrics | 🟢 92 | 🟢 92 |
13 other groups are unchanged.
🛡️ Audits
| 🔌 Plugin | 🛡️ Audit | 📏 Previous value | 📏 Current value | 🔄 Value change |
|---|---|---|---|---|
| JSDoc coverage | Functions coverage | 🟥 91 undocumented functions | 🟥 91 undocumented functions | |
| Code coverage | Function coverage | 🟩 97.3 % | 🟩 96.6 % | |
| Code coverage | Branch coverage | 🟨 82 % | 🟨 82.4 % | |
| Code coverage | Line coverage | 🟨 89.5 % | 🟨 89.7 % | |
| JSDoc coverage | Variables coverage | 🟥 15 undocumented variables | 🟥 16 undocumented variables |
439 other audits are unchanged.
13 other projects are unchanged.
342939b to
76b0616
Compare
BioPhoton
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice visual improvement of the logs!
Part of #888. Improves CI logs readability. Previously, the top-level logs from
cipackage were easy to miss among all the process outputs. To provide a better visual structure of what's happening, I prefixed these lines with a colored logo and ensured these logs are separated by blank lines. I think these make them pop out more.Before
After