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

Issue 338600043: [plaso] Switched from coveralls to codecov for code coverage analysis (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
6 years ago by Joachim Metz
Modified:
6 years ago
Reviewers:
onager
CC:
kiddi, log2timeline-dev_googlegroups.com, romaing
Visibility:
Public.

Description

[plaso] Switched from coveralls to codecov for code coverage analysis Depends on: https://codereview.appspot.com/334610043/

Patch Set 1 : Switched from coveralls to codecov for code coverage analysis #

Total comments: 1
Unified diffs Side-by-side diffs Delta from patch set Stats (+12 lines, -8 lines) Patch
M .travis.yml View 1 chunk +1 line, -1 line 0 comments Download
M config/dpkg/changelog View 1 chunk +2 lines, -2 lines 0 comments Download
M config/travis/install.sh View 2 chunks +2 lines, -4 lines 0 comments Download
M plaso/__init__.py View 1 chunk +1 line, -1 line 0 comments Download
M tests/cli/log2timeline_tool.py View 2 chunks +6 lines, -0 lines 1 comment Download

Messages

Total messages: 5
Joachim Metz
6 years ago (2018-03-20 21:04:30 UTC) #1
Joachim Metz
Code updated.
6 years ago (2018-03-20 21:12:45 UTC) #2
Joachim Metz
https://codereview.appspot.com/338600043/diff/20001/tests/cli/log2timeline_tool.py File tests/cli/log2timeline_tool.py (right): https://codereview.appspot.com/338600043/diff/20001/tests/cli/log2timeline_tool.py#newcode346 tests/cli/log2timeline_tool.py:346: @unittest.skipIf( changes that leaked into this CL
6 years ago (2018-03-20 21:17:32 UTC) #3
onager
LGTM
6 years ago (2018-03-24 12:55:29 UTC) #4
Joachim Metz
6 years ago (2018-03-24 13:42:28 UTC) #5
Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: review.py close codecov
Sign in to reply to this message.

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