Skip to content

Implement Quote effect#1346

Open
lionel- wants to merge 1 commit into
oak-nse/7-effects-refactorfrom
oak-nse/8-quote-effect
Open

Implement Quote effect#1346
lionel- wants to merge 1 commit into
oak-nse/7-effects-refactorfrom
oak-nse/8-quote-effect

Conversation

@lionel-

@lionel- lionel- commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Progress towards #1338
Branched from #1345

Adds ArgumentEffect::Quote for unevaluated arguments, e.g. quote(expr) or bquote(expr).

A quoted argument is skipped entirely by the scan and walk. quote({ x <- 1; library(foo); print(x) }) produces no uses, no bindings, no effects of any kind.

Unquote holes are handled in a future PR. I haven't looked into tidyeval yet.

Positron Release Notes

New Features

  • Smarter handling of quote() and bquote()

Bug Fixes

  • N/A

@lionel-
lionel- force-pushed the oak-nse/7-effects-refactor branch from e58369b to f152897 Compare July 17, 2026 08:56
@lionel-
lionel- force-pushed the oak-nse/8-quote-effect branch from b08c48f to ebc86c2 Compare July 17, 2026 08:56
@lionel-
lionel- force-pushed the oak-nse/7-effects-refactor branch from f152897 to f6eddb4 Compare July 17, 2026 11:26
@lionel-
lionel- force-pushed the oak-nse/8-quote-effect branch from ebc86c2 to 0f74b9b Compare July 17, 2026 11:26
@lionel-
lionel- force-pushed the oak-nse/7-effects-refactor branch from f6eddb4 to bd44ef0 Compare July 17, 2026 13:55
@lionel-
lionel- force-pushed the oak-nse/8-quote-effect branch from 0f74b9b to 5845e0c Compare July 17, 2026 13:55
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