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

Add 30 min timeout for tests #4416

Conversation

k-boikov
Copy link
Contributor

Background

image

Changes

Added timeout-minutes: 30 for the normal and docker tests.

PR Quality Checklist

  • My pull request is atomic and focuses on a single change.
  • I have thoroughly tested my changes with multiple different prompts.
  • I have considered potential risks and mitigations for my changes.
  • I have documented my changes clearly and comprehensively.
  • I have not snuck in any "extra" small tweaks changes.
  • I have run black . and isort . against my code to ensure it passes our linter.

@vercel
Copy link

vercel bot commented May 25, 2023

Deployment failed with the following error:

Resource is limited - try again in 6 minutes (more than 100, code: "api-deployments-free-per-day").

@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Patch and project coverage have no change.

Comparison is base (ddf5927) 68.46% compared to head (5458e97) 68.46%.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #4416   +/-   ##
=======================================
  Coverage   68.46%   68.46%           
=======================================
  Files          72       72           
  Lines        3494     3494           
  Branches      534      534           
=======================================
  Hits         2392     2392           
  Misses        930      930           
  Partials      172      172           

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@k-boikov k-boikov added the ci label May 25, 2023
@waynehamadi waynehamadi merged commit d3fc8c4 into Significant-Gravitas:master May 25, 2023
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

2 participants