Skip to content

Conversation

evanglass
Copy link
Contributor

@evanglass evanglass commented Jul 14, 2025

Fixes #2032

@evanglass evanglass marked this pull request as draft July 14, 2025 23:44
@evanglass evanglass closed this Jul 15, 2025
@evanglass evanglass reopened this Jul 15, 2025
@evanglass evanglass marked this pull request as ready for review July 15, 2025 17:01
@cpsievert cpsievert requested a review from karangattu July 15, 2025 17:12
@karangattu
Copy link
Collaborator

@evanglass thank you for adding this param. Can you add an entry about this to the changelog

@evanglass
Copy link
Contributor Author

Sure. I put it under improvements, let me know if that's good.

Copy link
Collaborator

@karangattu karangattu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Thanks

@evanglass evanglass marked this pull request as draft July 15, 2025 19:36
@evanglass evanglass marked this pull request as ready for review July 15, 2025 19:37
@evanglass
Copy link
Contributor Author

Sorry, I thought I accidentally wrote the wrong PR number in the changelog for a second cause I was looking at the issue. It looks good.

@karangattu karangattu enabled auto-merge (squash) July 15, 2025 19:44
@karangattu karangattu merged commit 3055517 into posit-dev:main Jul 16, 2025
151 of 154 checks passed
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.

[Feature]: Add timeout parameter to create_app_fixture
2 participants