Skip to content

Reduce the number of concurrent testruns in cibuild #1247

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

Merged
merged 1 commit into from
Jan 23, 2023
Merged

Conversation

bkoelman
Copy link
Member

@bkoelman bkoelman commented Jan 20, 2023

Closes #1241.

I've run the AppVeyor cibuild of this PR 10 times, spread throughout the day.
They were all green each time, so this seems to fix the issue.

QUALITY CHECKLIST

  • Changes implemented in code
  • Complies with our contributing guidelines
  • N/A: Adapted tests
  • N/A: Documentation updated

@codecov
Copy link

codecov bot commented Jan 20, 2023

Codecov Report

Merging #1247 (7c5e7cd) into master (be99234) will not change coverage.
The diff coverage is n/a.

❗ Current head 7c5e7cd differs from pull request most recent head 9b6fac9. Consider uploading reports for the commit 9b6fac9 to get more accurate results

@@           Coverage Diff           @@
##           master    #1247   +/-   ##
=======================================
  Coverage   92.62%   92.62%           
=======================================
  Files         243      243           
  Lines        7795     7795           
=======================================
  Hits         7220     7220           
  Misses        575      575           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@bkoelman bkoelman marked this pull request as ready for review January 21, 2023 01:02
@bkoelman bkoelman requested a review from maurei January 21, 2023 01:02
@maurei maurei merged commit c7e396d into master Jan 23, 2023
@maurei maurei deleted the tests-throttle branch January 23, 2023 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

AppVeyor build sometimes fails with PostgreSQL timeout
2 participants