Skip to content
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

Remove the --warm-coverage-cache option cause by issue #4600 #5911

Conversation

kayw-geek
Copy link

Problem Description

This PR addresses the issue where using the --warm-code-coverage-cache option results in the following error message:

Cache for static analysis has not been configured.

Background

In issue #4600, the --coverage-cache <dir> option was removed. However, the current implementation still checks for the configuration of --coverage-cache <dir> when the --warm-code-coverage-cache option is used, leading to the error message if the cache is not configured.

Solution

This PR resolves the issue by removing the outdated check for --coverage-cache <dir> configuration in the --warm-code-coverage-cache option handling. This will prevent the error message from appearing and ensure normal operation for future users.

References


@sebastianbergmann
Copy link
Owner

Thank you for your contribution. I appreciate the time you invested in preparing this pull request. However, I have decided not to merge it. The functionality in question still exists and its removal is not intended.

The CLI option for configuring the cache directory has been removed. It must be configured using the XML configuration file.

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

Successfully merging this pull request may close these issues.

2 participants