Closed PietrH closed 5 months ago
@PietrH do you expect me to some in depth tests ? if so which ? or just approve & merge ?
If it looks good you can approve, in general I prefer if the one who opens a PR/is assigned also does the merge.
In this case: see if you like the output of citation()
, see if you agree with the changes I did to DESCRIPTION
, what do you think about the codemeta project?
I'll take a look
Op ma 12 feb. 2024 11:07 schreef Pieter Huybrechts @.***
:
If it looks good you can approve, in general I prefer if the one who opens a PR/is assigned also does the merge.
In this case: see if you like the output of citation(), see if you agree with the changes I did to DESCRIPTION, what do you think about the codemeta project?
— Reply to this email directly, view it on GitHub https://github.com/inbo/iassetR/pull/36#issuecomment-1938370435, or unsubscribe https://github.com/notifications/unsubscribe-auth/AEAAQOCA3SJR5TZAR32GRBTYTHSVBAVCNFSM6AAAAABDBHTWZOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMZYGM3TANBTGU . You are receiving this because your review was requested.Message ID: @.***>
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 0.00%. Comparing base (
b912fb1
) to head (96a84b5
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Fixes #29
Also add a bugreports field to the description and a codemeta file