Closed ThomUK closed 11 months ago
1) A reference to the issue addressed by this pull-request: closes #82 closes #75 also does enabling work for #28, but does not complete it
2) A description of the changes proposed in this PR:
3) Add @mentions for the people who will review this PR: @francisbarton I'd appreciate a review of this one...
devtools::test()
suppressMessages()
Thank you for contributing to {SPCreporter}!!
Cancelling this and merging to dev branch first...
Please complete the following information:
1) A reference to the issue addressed by this pull-request:
closes #82 closes #75 also does enabling work for #28, but does not complete it
2) A description of the changes proposed in this PR:
3) Add @mentions for the people who will review this PR:
@francisbarton I'd appreciate a review of this one...
Please confirm that you have:
devtools::test()
and fixed all failing tests and warnings.suppressMessages()
to any test message which breaks the test progress UI.Thank you for contributing to {SPCreporter}!!