Skip to content

Comments

ci: increase timeout from '12' to '15' minutes to avoid cancel errors#1526

Merged
zimeg merged 2 commits intomainfrom
zimeg-ci-timeout
Nov 3, 2025
Merged

ci: increase timeout from '12' to '15' minutes to avoid cancel errors#1526
zimeg merged 2 commits intomainfrom
zimeg-ci-timeout

Conversation

@zimeg
Copy link
Member

@zimeg zimeg commented Oct 31, 2025

This PR extends the test timeout from "12" to "15" minutes to avoid frequent timeouts on main and in PRs:

🔗 https://github.com/slackapi/java-slack-sdk/actions/runs/18954549078/job/54127848354

Category

  • Maintenance

Notes

We might want to consider running these tests in parallel instead of increasing the timeout going forward since this isn't an ideal approach IMHO. For now I am hoping to unblock CI 🤖

Requirements

Please read the Contributing guidelines and Code of Conduct before creating this issue or pull request. By submitting, you agree to those rules.

@zimeg zimeg self-assigned this Oct 31, 2025
@zimeg zimeg added the github_actions Pull requests that update GitHub Actions code label Oct 31, 2025
@codecov
Copy link

codecov bot commented Nov 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.91%. Comparing base (f681eec) to head (0d301aa).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1526      +/-   ##
============================================
- Coverage     72.92%   72.91%   -0.01%     
  Complexity     4403     4403              
============================================
  Files           476      476              
  Lines         14036    14036              
  Branches       1471     1471              
============================================
- Hits          10236    10235       -1     
- Misses         2928     2930       +2     
+ Partials        872      871       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Contributor

@WilliamBergamin WilliamBergamin left a comment

Choose a reason for hiding this comment

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

Nice!!

@zimeg
Copy link
Member Author

zimeg commented Nov 3, 2025

@WilliamBergamin Kind thanks! I'm rerunning these tests but will merge this once these pass ⏳ ✨

@zimeg zimeg enabled auto-merge (squash) November 3, 2025 19:05
@zimeg zimeg merged commit 0a30aca into main Nov 3, 2025
6 of 8 checks passed
@zimeg zimeg deleted the zimeg-ci-timeout branch November 3, 2025 20:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

github_actions Pull requests that update GitHub Actions code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants