Skip to content

Conversation

@weebl2000
Copy link
Contributor

500ms socket timeout is incredibly low, 5 seconds makes more sense. Kept the total timeout the same.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

@weebl2000 weebl2000 force-pushed the increase-read-timeout branch 2 times, most recently from b9aac32 to 81fbd9d Compare December 8, 2025 17:28
500ms socket timeout is incredibly low
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated no new comments.

@jamesarich jamesarich enabled auto-merge December 8, 2025 20:08
@codecov
Copy link

codecov bot commented Dec 8, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 0.52%. Comparing base (9246b3b) to head (aa370b0).
⚠️ Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
...m/geeksville/mesh/repository/radio/TCPInterface.kt 0.00% 2 Missing ⚠️
Additional details and impacted files
@@          Coverage Diff          @@
##            main   #3929   +/-   ##
=====================================
  Coverage   0.52%   0.52%           
=====================================
  Files        394     394           
  Lines      23136   23136           
  Branches    2933    2933           
=====================================
  Hits         122     122           
  Misses     22993   22993           
  Partials      21      21           

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

@jamesarich
Copy link
Collaborator

@weebl2000 i went ahead and fixed those detekt complaints, it looked like they had been previously suppressed/baselined (were not introduced by your changes, just were being hidden).

For future contributions please ensure that the code passes ./gradlew spotless detekt checks. Will save us from having to ask (and some ci churn). Thanks!

@jamesarich jamesarich added this pull request to the merge queue Dec 8, 2025
@weebl2000
Copy link
Contributor Author

@weebl2000 i went ahead and fixed those detekt complaints, it looked like they had been previously suppressed/baselined (were not introduced by your changes, just were being hidden).

For future contributions please ensure that the code passes ./gradlew spotless detekt checks. Will save us from having to ask (and some ci churn). Thanks!

Will keep it in mind 👌🏽

Merged via the queue into meshtastic:main with commit 2ad703c Dec 8, 2025
6 checks passed
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.

2 participants