Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix repeated message / group issue (closes #62) #64

Merged
merged 2 commits into from Feb 5, 2020

Conversation

@eddelbuettel
Copy link
Owner

eddelbuettel commented Feb 5, 2020

With thanks to @sircosine spotting the issue and fix

@eddelbuettel eddelbuettel merged commit 2a00e03 into master Feb 5, 2020
2 checks passed
2 checks passed
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
@eddelbuettel eddelbuettel deleted the feature/repeated_message_fix branch Feb 5, 2020
@sircosine

This comment has been minimized.

Copy link

sircosine commented Feb 5, 2020

Thank you for getting fix in quickly!

@eddelbuettel

This comment has been minimized.

Copy link
Owner Author

eddelbuettel commented Feb 5, 2020

No problem -- thanks for the heads-up and excellent fix!

I plan to do two things I have done for other packages of mine: switch to tinytest, and use pre-built vignettes. Hopefully on the weekend, and a new and improved RProtoBuf should then be on CRAN by next week.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.