Skip to content

Unexport unused s3 methods#147

Merged
MichaelChirico merged 3 commits intomainfrom
unexport-unused-s3-methods
Jan 31, 2025
Merged

Unexport unused s3 methods#147
MichaelChirico merged 3 commits intomainfrom
unexport-unused-s3-methods

Conversation

@MichaelChirico
Copy link
Copy Markdown
Collaborator

Part of #76.

Unfortunately, I've learned that some primitives, e.g. -, don't appear on the call stack when checked from a corresponding method:

`-.foo` = function(x) sys.calls()
`-`(structure(1L, class='foo'))
# [[1]]
# `-.foo`(structure(1L, class = "foo"))

Therefore I'll need to simply un-export these ones directly. I'll do so in a separate PR.

@MichaelChirico MichaelChirico merged commit 1da901a into main Jan 31, 2025
@MichaelChirico MichaelChirico deleted the unexport-unused-s3-methods branch January 31, 2025 06:09
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.

1 participant