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

Understand setReplaceMethod() #266

Closed
thomasp85 opened this issue Aug 4, 2014 · 1 comment
Closed

Understand setReplaceMethod() #266

thomasp85 opened this issue Aug 4, 2014 · 1 comment

Comments

@thomasp85
Copy link
Member

@thomasp85 thomasp85 commented Aug 4, 2014

It seems in my use case that roxygen behaves inconsistently when documenting S4 methods. As advised in Advanced R Programming I usually document the generic and will then reference this documentation in the methods. This works fine for setMethod(), but fails with setReplaceMethod. In the latter roxygen fails to create any of the needed aliases, and furthermore need a @name declarative for roxygen to not throw an error.

Documentation of these two cases should be equal IMHO

best

Thomas

@hadley hadley closed this in 2b6fe4a Aug 29, 2014
@hadley
Copy link
Member

@hadley hadley commented Aug 29, 2014

Done, although I don't really understand the point of setReplaceMethod("foo") since it's more letters than setMethod("foo<-")

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

Successfully merging a pull request may close this issue.

None yet
2 participants