- Notifications
You must be signed in to change notification settings - Fork1k
chore: skip Gemini tests in local runs#2181
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
PR Change SummaryUpdated testing procedures to improve local development experience by skipping certain tests and clarifying test execution status.
Modified Files
How can I customize these reviews?Check out theHyperlint AI Reviewer docs for more information on how to customize the review. If you just want to ignore it on this PR, you can add the Note specifically for link checks, we only check the first 30 links in a file and we cache the results for several hours (for instance, if you just added a page, you might experience this). Our recommendation is to add |
Skip Gemini example tests in local runs to prevent it failing when theuser has gcloud configured but not with the ability to run the GeminiAPIs as that prevents coverage profiles from being run when developingnew features or fixing bugs.Switch test skip from a print to pytest.skip so that its clear that thetest didn't actually run.Fixes:pydantic#2167
ee3a9ee
to3638656
Compare9b1a89b
intopydantic:mainUh oh!
There was an error while loading.Please reload this page.
Skip Gemini example tests in local runs to prevent it failing when the user has gcloud configured but not with the ability to run the Gemini APIs as that prevents coverage profiles from being run when developing new features or fixing bugs.
Switch test skip from a print to pytest.skip so that its clear that the test didn't actually run.
Fixes:#2167