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

@family refers to name only, not all aliases, #283 #367

Merged
merged 1 commit into from Oct 7, 2015
Merged

@family refers to name only, not all aliases, #283 #367

merged 1 commit into from Oct 7, 2015

Conversation

@gaborcsardi
Copy link
Collaborator

@gaborcsardi gaborcsardi commented Aug 5, 2015

I had this PR lying around. Details were discussed at #283.

I am actually not convinced that this is always the good behavior. Sometimes it makes sense to link to all aliases, so if you have an idea on how to make this user configurable, per family, probably, I can update this PR.

@hadley
Copy link
Member

@hadley hadley commented Oct 4, 2015

Yeah, we need to think this one through

@hadley
Copy link
Member

@hadley hadley commented Oct 6, 2015

Could you please add a bullet to NEWS? I think I just need to try it out on a few of my projects and see how it feels.

@gaborcsardi
Copy link
Collaborator Author

@gaborcsardi gaborcsardi commented Oct 7, 2015

OK, NEWS updated.

@hadley hadley merged commit 01eaca7 into r-lib:master Oct 7, 2015
1 check passed
@hadley
Copy link
Member

@hadley hadley commented Oct 7, 2015

Thanks! I definitely prefer the new behaviour for roxygen2 itself

@gaborcsardi gaborcsardi mentioned this pull request Sep 19, 2018
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 issues

Successfully merging this pull request may close these issues.

None yet

2 participants