Skip to content

Conversation

eddelbuettel
Copy link
Member

Simple pull request to add Rcpp::message() as discussed in #1145

Checklist

  • Code compiles correctly
  • R CMD check still passes all tests
  • Prefereably, new tests were added which fail without the change
  • Document the changes by file in ChangeLog

@codecov-io
Copy link

codecov-io commented Mar 21, 2021

Codecov Report

Merging #1146 (badf27a) into master (a9e9bda) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1146   +/-   ##
=======================================
  Coverage   97.44%   97.44%           
=======================================
  Files          64       64           
  Lines        2783     2783           
=======================================
  Hits         2712     2712           
  Misses         71       71           

@eddelbuettel
Copy link
Member Author

Quick poke just to say that I think objections have been addressed to I suggest to proceed and merge later. Please holler if you disagree...

@eddelbuettel
Copy link
Member Author

Thanks guys!

@eddelbuettel eddelbuettel merged commit d370c5b into master Mar 22, 2021
@eddelbuettel eddelbuettel deleted the feature/message branch March 22, 2021 22:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants