Skip to content

Conversation

@CommitToday
Copy link

Overview

This pull request addresses issue #4635 by adding alternative executable names for the DotCover tool. The change allows the tool to be found even if it's installed with different naming conventions (e.g., lowercase or without the .exe extension).

Checklist

  • Code changes have been reviewed
  • Tests have been updated to reflect the changes
  • Documentation has been updated if necessary

Proof

The changes modify the GetToolExecutableNames method in DotCoverTool.cs to include additional executable names (dotCover and dotcover) alongside the existing dotCover.exe. This ensures the tool can be located regardless of how it is installed on the system.

Closes #4635

@gamingrobot
Copy link

gamingrobot commented Nov 12, 2025

This does not fix #4635

When using the correct tool syntax #tool dotnet:?package=JetBrains.dotCover.CommandLineTools&version=2025.2.3 instead of #tool "nuget:?package=JetBrains.dotCover.CommandLineTools&version=2025.2.3" cake correctly finds the tool executable.

The issue is in how cake is formatting the arguments to dotCover for versions 2025.2+.

@CommitToday
Copy link
Author

@microsoft-github-policy-service agree

@devlead devlead changed the base branch from develop to feature/dotcover December 5, 2025 22:44
@devlead devlead force-pushed the fix-issue-4635-dotcover-no-longer-works-with-version-2025-2-of-je branch from 0780b50 to 5459fe8 Compare December 5, 2025 22:45
@devlead
Copy link
Member

devlead commented Dec 5, 2025

This does not fix #4635

When using the correct tool syntax #tool dotnet:?package=JetBrains.dotCover.CommandLineTools&version=2025.2.3 instead of #tool "nuget:?package=JetBrains.dotCover.CommandLineTools&version=2025.2.3" cake correctly finds the tool executable.

The issue is in how cake is formatting the arguments to dotCover for versions 2025.2+.

correct, together with #4628 it should

@devlead devlead merged commit 99d957c into cake-build:feature/dotcover Dec 5, 2025
1 of 4 checks passed
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.

DotCover no longer works with version 2025.2+ of JetBrains.dotCover.CommandLineTools

3 participants