Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(3)

Issue 12684043: code review 12684043: misc/emacs: Add support for code coverage output of go test (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
11 years, 7 months ago by Dominik Honnef
Modified:
11 years, 7 months ago
Reviewers:
bradfitz
CC:
adonovan, bradfitz, golang-dev
Visibility:
Public.

Description

misc/emacs: Add support for code coverage output of go test Renders code coverage as an overlay, replicating the look of the HTML that go tool cover produces. Also some cleanups.

Patch Set 1 #

Patch Set 2 : diff -r 3c3b4d0d3e15 https://code.google.com/p/go #

Patch Set 3 : diff -r 3c3b4d0d3e15 https://code.google.com/p/go #

Unified diffs Side-by-side diffs Delta from patch set Stats (+97 lines, -15 lines) Patch
M misc/emacs/go-mode.el View 1 7 chunks +97 lines, -15 lines 0 comments Download

Messages

Total messages: 3
Dominik Honnef
Hello adonovan@google.com (cc: golang-dev@googlegroups.com), I'd like you to review this change to https://code.google.com/p/go
11 years, 7 months ago (2013-08-08 22:49:43 UTC) #1
bradfitz
LGTM On Thu, Aug 8, 2013 at 3:49 PM, <dominik.honnef@gmail.com> wrote: > Reviewers: adonovan, > ...
11 years, 7 months ago (2013-08-08 22:54:25 UTC) #2
bradfitz
11 years, 7 months ago (2013-08-08 22:54:28 UTC) #3
*** Submitted as https://code.google.com/p/go/source/detail?r=3cc189fc323c ***

misc/emacs: Add support for code coverage output of go test

Renders code coverage as an overlay, replicating the look of the
HTML that go tool cover produces.

Also some cleanups.

R=adonovan, bradfitz
CC=golang-dev
https://codereview.appspot.com/12684043

Committer: Brad Fitzpatrick <bradfitz@golang.org>
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b