Skip to content

Update test_study.py #244

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Update test_study.py #244

wants to merge 1 commit into from

Conversation

TLSDC
Copy link
Collaborator

@TLSDC TLSDC commented May 1, 2025

Description by Korbit AI

What change is being made?

Add a pytest skip marker to the test_launch_parallel_study test case, with a reason indicating the use case is not relevant at the moment.

Why are these changes being made?

The current relevance of the test_launch_parallel_study use case has diminished, implying that running this test is not necessary at this time. This approach allows the development and testing process to remain focused on more relevant features and tests while preserving the ability to revisit this test case in the future.

Is this description stale? Ask me to generate a new description by commenting /korbit-generate-pr-description

Copy link

@korbit-ai korbit-ai bot left a comment

Choose a reason for hiding this comment

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

I've completed my review and didn't find any issues... but I did find this penguin.

 __
( o>
///\
\V_/_

Check out our docs on how you can make Korbit work best for you and your team.

Loving Korbit!? Share us on LinkedIn Reddit and X

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