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

Update Roxygen comments (examples) #129

Merged
merged 6 commits into from
Jan 26, 2023

Conversation

eitsupi
Copy link
Contributor

@eitsupi eitsupi commented Jan 25, 2023

Related to #128

Most of the examples looked fine to me to run.
I've wrapped only the ones that are meaningless unless they are executed interactively with if (interactive()).
(This is a way used by shinyWidgets, for example. https://github.com/dreamRs/shinyWidgets/blob/f37e59a33630efc358e9b317207643c99f67d4df/R/input-autonumeric.R#L53-L89)

Feel free to edit this branch if you find anything inappropriate.

R/httpgd.R Show resolved Hide resolved
@eitsupi eitsupi marked this pull request as ready for review January 25, 2023 15:12
@nx10
Copy link
Owner

nx10 commented Jan 25, 2023

Looks excellent, thank you!

R/httpgd.R Outdated Show resolved Hide resolved
@nx10
Copy link
Owner

nx10 commented Jan 25, 2023

Also: Add yourself to the list of contributors inside the DESCRIPTION (if you want to) 🚀

@nx10 nx10 merged commit 2295c89 into nx10:bh-hotfix Jan 26, 2023
@eitsupi eitsupi deleted the update-roxygen-comments branch January 27, 2023 10:42
kar9222 pushed a commit to kar9222/httpgd that referenced this pull request Apr 22, 2023
* update examples not to use `dontrun` in Rd files

* update hgd_id usage

* add missing @return comment

* remove unnecessary hgd() and dev.off()

* add a new contributor

* remove tmpfiles on exit
@eitsupi eitsupi mentioned this pull request Jan 21, 2024
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