Skip to content
This repository has been archived by the owner on Jun 12, 2020. It is now read-only.

fxn naming scheme #6

Closed
sckott opened this issue Oct 26, 2016 · 2 comments
Closed

fxn naming scheme #6

sckott opened this issue Oct 26, 2016 · 2 comments

Comments

@sckott
Copy link

sckott commented Oct 26, 2016

Something we've been doing in some of our pkgs, is to prefix fxns with something very short, e.g., here could do ukp or up - so get_neighbourhoods could become ukp_neighbourhoods

because R doesn't require users to namespace fxn calls, fxn collisions can happen, so this helps that

and helps users with knowing what pkg a fxn comes from easily

just a thought...

@njtierney
Copy link
Owner

This is a great idea, thanks @sckott ! @masalmon has also suggested this, it is a nice user feature

njtierney added a commit that referenced this issue Oct 27, 2016
Also addressed changes in issues #6 and #7, thanks @sckott!
@njtierney
Copy link
Owner

fixed in the above commit. Thanks again for your suggestion!

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants