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

Roxygensed the documentation #4

Merged
merged 2 commits into from
Jan 29, 2015
Merged

Roxygensed the documentation #4

merged 2 commits into from
Jan 29, 2015

Conversation

karthik
Copy link
Member

@karthik karthik commented Jan 29, 2015

Okay. I'll drink the Kool-aid.

Awesome! 👍

I do see the utility in setting up tests, and have added some preliminary function tests. I also made some small function tweaks that format location values correctly and throw errors when you try to hand it a non-existent location. I'll add a plot function and a quick data manipulation function (form interaction matrix from edgelist output) sometime soon-ish.

Cool! 💯

I think what I'll need your help with the most is the conversion of my current documentation setup to the roxygen setup. Also, please continue to point out things I should be doing (writing tests, etc.).

Done. See the inline docs in this PR. I'll provide feedback on tests on the next round.

taddallas pushed a commit that referenced this pull request Jan 29, 2015
Roxygensed the documentation
@taddallas taddallas merged commit bf764f9 into master Jan 29, 2015
@karthik karthik deleted the roxygen branch January 29, 2015 20:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants