Fix instances where error for "lost braces" appear#276
Open
chitrams wants to merge 1 commit intoflorianhartig:masterfrom
Open
Fix instances where error for "lost braces" appear#276chitrams wants to merge 1 commit intoflorianhartig:masterfrom
chitrams wants to merge 1 commit intoflorianhartig:masterfrom
Conversation
Owner
|
Dear Chitra, thanks for this - I realize I'm behind fixing various issues with this package, which I didn't manage simply due to time reasons. I'll try to draft a CRAN update to BT as soon as possible, where these issues will be solved. Best |
Owner
|
That being said - there seem to be issues, I guess the describe / item syntax requires a { to follow \item ? I can fix this myself though! |
Author
|
Dear Florian, No rush! We did this as part of R Dev Day Australia and hoped this would be helpful :) I will check that and update the PR! Cheers, |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR fixes instances where the "Lost braces" errors appear. The changes made were:
itemize{}todescribe{}") where they weren't needed and led to errorsBackground
This task was proposed by R Core developer Kurt Hornik as part of an initiative to help package authors by submitting pull requests to public repositories that exhibit easily resolvable CRAN check issues, starting with “Lost braces” in documentation files (issue here)
Next steps
If this package uses roxygen2, you may wish to regenerate documentation to ensure future updates do not reintroduce these issues. Otherwise, the corrected .Rd files can be merged directly.
Hope this is helpful! :)