-
Notifications
You must be signed in to change notification settings - Fork 16
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. |
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 👍
@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 <[email protected]>
This PR includes:
utils
for runner result cachingexecutePlugin
andexecutePlugins
CacheConfig
tomodels
package and use it incore
packageRelated:
#1048