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

Align snapshot tests #6987

Merged
merged 2 commits into from
Feb 13, 2024
Merged

Align snapshot tests #6987

merged 2 commits into from
Feb 13, 2024

Conversation

krlmlr
Copy link
Member

@krlmlr krlmlr commented Feb 4, 2024

Changes in cli, perhaps?

@DavisVaughan DavisVaughan merged commit cf45725 into main Feb 13, 2024
11 checks passed
@DavisVaughan
Copy link
Member

cli I think, in this case, thanks!

@DavisVaughan DavisVaughan deleted the b-snapshot branch February 13, 2024 21:31
DavisVaughan added a commit that referenced this pull request Feb 13, 2024
* Align snapshot tests

* Bump rlang and cli for snapshot tests

---------

Co-authored-by: Davis Vaughan <davis@posit.co>
DavisVaughan added a commit that referenced this pull request Feb 13, 2024
* Test that `summarize()` strips off the subclass

* Align snapshot tests (#6987)

* Align snapshot tests

* Bump rlang and cli for snapshot tests

---------

Co-authored-by: Davis Vaughan <davis@posit.co>

* Tweak the test a little

* Mention `summarise()` / `reframe()` behavior in extension docs

---------

Co-authored-by: Davis Vaughan <davis@posit.co>
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