Skip to content

Fix context construction and generic bindings with development S7 - #240

Merged
gadenbuie merged 4 commits into
posit-dev:mainfrom
t-kalinowski:fix/s7-compatibility-preview-20261012
Nov 2, 2026
Merged

gadenbuie merged 4 commits into
posit-dev:mainfrom
t-kalinowski:fix/s7-compatibility-preview-20261012

Conversation

@t-kalinowski

Copy link
Copy Markdown
Member

Empty context from btw() fails when its empty string is forwarded positionally to the parent constructor, and NEWS extraction fails when as.character remains an S7 external-generic sentinel. Name the text argument and clean external generic bindings after method registration.

Use S7’s load, unload, and build hooks when available, retaining methods_register() for older releases. On development S7, unloading removes the package’s external S7 methods and dependency hooks, and the build hook cleans external generic bindings after method declarations.

@t-kalinowski

Copy link
Copy Markdown
Member Author

Thanks for maintaining btw! I'm planning to submit the next version of S7 to CRAN on November 16 (see RConsortium/S7#786).

Would you be able to merge these compatibility changes and submit an updated version of btw to CRAN before then? Please let me know if you need more time or if there’s anything I can help with.

@gadenbuie
gadenbuie merged commit bdd8d1b into posit-dev:main Nov 2, 2026
9 checks passed
@gadenbuie

Copy link
Copy Markdown
Collaborator

Thanks @t-kalinowski -- I'll try to make the Nov 16 deadline

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