Skip to content

Fix coverage for kotl files - #1097

Closed
matsl wants to merge 1 commit into
masterfrom
pr_fix_coverage_on_subfolder_files
Closed

matsl wants to merge 1 commit into
masterfrom
pr_fix_coverage_on_subfolder_files

Conversation

@matsl

@matsl matsl commented Nov 14, 2026

Copy link
Copy Markdown
Collaborator

What

Files in subfolders, i.e. kotl, were not instrumented. Use an absolute
file name to point testcover-start to the right buffer.

Added a local key binding for testcover-next-mark.

  • test/hy-test-coverage.el (hy-test-coverage-file): Use absolute
    filename. Add keybinding for testcover-next-mark.

Why

Running make coverage on kotl files did not work due to reason given
above. This fixes it an adds a key binding to ease working with the
resulting splotches.

Files in subfolders, i.e. kotl, were not instrumented. Use an absolute
file name to point testcover-start to the right buffer.

Added a local key binding for testcover-next-mark.

* test/hy-test-coverage.el (hy-test-coverage-file): Use absolute
filename. Add keybinding for testcover-next-mark.
@matsl matsl self-assigned this Nov 14, 2026
@matsl
matsl requested review from rswgnu and removed request for rswgnu November 14, 2026 22:17
@matsl matsl closed this Nov 14, 2026
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.

1 participant