search

AdamNiederer / cov

An emacs extension for displaying coverage data on your code
GNU General Public License v3.0
76 stars 16 forks source link

Coverage cleanup #19

Closed xendk closed 6 years ago

xendk commented 6 years ago

Clean out coverages when not in use.

Haven't gotten round to watching the coverage file for changes, got sidetracked by undercover.el reporting odd coverage for cov.el. Need to look into that.

xendk commented 6 years ago

Or maybe it's me that's screwing up... This looks right, doesn't it? https://codecov.io/gh/xendk/cov/src/coverage/cov.el

I'll look into it when I'm more awake.

xendk commented 6 years ago

Stupid me. I'd forgotten the fact that undercover.el merges new coverage with an existing file.

AdamNiederer commented 6 years ago

I'm curious about the motivation behind using a timer to clean up our stored coverage data - wouldn't kill-buffer-hook be more efficient while achieving the same effect?

xendk commented 6 years ago

I think it was the lateness in the day that made it seem like a good idea. I'll rework it.

xendk commented 6 years ago

There, much better.

AdamNiederer commented 6 years ago

Looks good. Thanks!