-
Notifications
You must be signed in to change notification settings - Fork 15
feat: add audit output caching for execute plugin #1057
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
Code PushUp🤨 Code PushUp report has both improvements and regressions – compared current commit f8eec51 with previous commit ae52e3b. 🕵️ See full comparison in Code PushUp portal 🔍 🏷️ Categories👍 2 groups improved, 👎 2 groups regressed, 👍 4 audits improved, 👎 5 audits regressed, 13 audits changed without impacting score🗃️ Groups
17 other groups are unchanged. 🛡️ Audits
588 other audits are unchanged. |
matejchalk
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.
Thanks a lot for the fast PR 👍
Co-authored-by: Matěj Chalk <34691111+matejchalk@users.noreply.github.com>
@code-pushup/ci
@code-pushup/cli
@code-pushup/create-cli
@code-pushup/models
@code-pushup/nx-plugin
@code-pushup/core
@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
commit: |
This PR depends on: - #1058 - #1057 This PR includes: - docs on [`--cache` options and caching](https://github.com/code-pushup/cli/pull/1059/files#diff-5357afeca9b96eccbc161aa37719ecf7668ac0ebae529e0695f2a9da5cf3b144) - docs on [how to cache with Nx](https://github.com/code-pushup/cli/pull/1059/files#diff-15986190ef9581ab59bcd5483b2c09e7fd0bd439d6f6cddbc94b0b1de094ee51) - docs on how to use with Turborepo - logic to respect cache options - e2e tests testing the full cache flow (inc `--skipReports`) Suggested followup PR would be to implement Nx caching. This is partially started here: #1035. > [!note] > Potential changes arrive after #1058 is merged Closes #1048 --------- Co-authored-by: Matěj Chalk <34691111+matejchalk@users.noreply.github.com>
This PR includes:
utilsfor runner result cachingexecutePluginandexecutePluginsCacheConfigtomodelspackage and use it incorepackageRelated:
#1048