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

Provide additional aliases for S4 subsetting methods #117

Merged
merged 1 commit into from
May 11, 2020

Conversation

eddelbuettel
Copy link
Contributor

@eddelbuettel eddelbuettel commented May 10, 2020

This suppresses an entirely harmless but utterly annoying warning that pops up every now and then. S4 methods are somewhere between wonderful, a can of worms, and a hornets. The change is this PR votes for last option. (Underlying cause may be a bug on roxygen2 as the warning comes and goes over rebuilds. Now it should be gone with a capital G.)

No code changes.

Suppresses this beauty:

checking Rd cross-references
W  checking for missing documentation entries (702ms)
   Undocumented S4 methods:
     generic '[' and siglist 'tiledb_array'
     generic '[' and siglist 'tiledb_config'
     generic '[' and siglist 'tiledb_dense'
     generic '[' and siglist 'tiledb_filter_list'
     generic '[' and siglist 'tiledb_sparse'
     generic '[<-' and siglist 'tiledb_array'
     generic '[<-' and siglist 'tiledb_config'
     generic '[<-' and siglist 'tiledb_dense'
     generic '[<-' and siglist 'tiledb_sparse'
   All user-level objects in a package (including S4 classes and methods)
   should have documentation entries.
   See chapterWriting R documentation filesin theWriting R
   Extensionsmanual.checking for code/documentation mismatches (2.1s)

@eddelbuettel eddelbuettel merged commit 972981f into master May 11, 2020
@eddelbuettel eddelbuettel deleted the bugfix/more_s4_aliases branch May 11, 2020 21:17
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