Bug 1862881 - Review Request: R-covr - Test Coverage for Packages
Summary: Review Request: R-covr - Test Coverage for Packages
Keywords:
Status: CLOSED ERRATA
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: Unspecified
OS: Unspecified
unspecified
unspecified
Target Milestone: ---
Assignee: José Matos
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On: 1862859
Blocks: 1823027
TreeView+ depends on / blocked
 
Reported: 2020-08-03 05:59 UTC by Elliott Sales de Andrade
Modified: 2020-08-17 01:13 UTC (History)
2 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2020-08-17 01:07:55 UTC
Type: ---
Embargoed:
jamatos: fedora-review+


Attachments (Terms of Use)

Description Elliott Sales de Andrade 2020-08-03 05:59:27 UTC
Spec URL: https://qulogic.fedorapeople.org/reviews/R-covr/R-covr.spec
SRPM URL: https://qulogic.fedorapeople.org/reviews/R-covr/R-covr-3.5.0-1.fc31.src.rpm

Description:
Track and report code coverage for your package and (optionally) upload the
results to a coverage service like 'Codecov' <http://codecov.io> or 'Coveralls'
<http://coveralls.io>. Code coverage is a measure of the amount of code being
exercised by a set of tests. It is an indirect measure of test quality and
completeness. This package is compatible with any testing methodology or
framework and tracks coverage of both R code and compiled C/C++/FORTRAN code.

Comment 1 José Matos 2020-08-05 18:48:04 UTC
For another déjà vu fedora-review says:

- Package have the default element marked as %%doc :doc, DESCRIPTION
- Package requires R-core.
- If your application is a C or C++ application you must list a
  BuildRequires against gcc, gcc-c++ or clang.
  Note: No gcc, gcc-c++ or clang found in BuildRequires
  See: https://docs.fedoraproject.org/en-US/packaging-guidelines/C_and_C++/

Repeating what I had said previously in other reviews it seems OK to follow the dependency from R-devel -> R-core-devel.

The license is correct and the spec is well written.

So the package is approved.

Comment 2 Gwyn Ciesla 2020-08-06 12:33:50 UTC
(fedscm-admin):  The Pagure repository was created at https://src.fedoraproject.org/rpms/R-covr

Comment 3 Fedora Update System 2020-08-08 23:58:35 UTC
FEDORA-2020-e67ab07606 has been submitted as an update to Fedora 32. https://bodhi.fedoraproject.org/updates/FEDORA-2020-e67ab07606

Comment 4 Fedora Update System 2020-08-09 00:02:04 UTC
FEDORA-2020-939748287f has been submitted as an update to Fedora 31. https://bodhi.fedoraproject.org/updates/FEDORA-2020-939748287f

Comment 5 Fedora Update System 2020-08-09 03:01:46 UTC
FEDORA-2020-939748287f has been pushed to the Fedora 31 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-939748287f \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-939748287f

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 6 Fedora Update System 2020-08-09 03:47:47 UTC
FEDORA-2020-e67ab07606 has been pushed to the Fedora 32 testing repository.
In short time you'll be able to install the update with the following command:
`sudo dnf install --enablerepo=updates-testing --advisory=FEDORA-2020-e67ab07606 \*`
You can provide feedback for this update here: https://bodhi.fedoraproject.org/updates/FEDORA-2020-e67ab07606

See also https://fedoraproject.org/wiki/QA:Updates_Testing for more information on how to test updates.

Comment 7 Fedora Update System 2020-08-17 01:07:55 UTC
FEDORA-2020-939748287f has been pushed to the Fedora 31 stable repository.
If problem still persists, please make note of it in this bug report.

Comment 8 Fedora Update System 2020-08-17 01:13:44 UTC
FEDORA-2020-e67ab07606 has been pushed to the Fedora 32 stable repository.
If problem still persists, please make note of it in this bug report.


Note You need to log in before you can comment on or make changes to this bug.