Skip to content

docs: remove code-review comment#1305

Merged
josecelano merged 1 commit intotorrust:developfrom
josecelano:1276-overhaul-core-tracker-review-announce-handler-peer-mutability
Feb 20, 2025
Merged

docs: remove code-review comment#1305
josecelano merged 1 commit intotorrust:developfrom
josecelano:1276-overhaul-core-tracker-review-announce-handler-peer-mutability

Conversation

@josecelano
Copy link
Copy Markdown
Member

We decided not to change it.

  • The returned value is simpler.
  • We force the initial peer to change so there is no confusion about what was the final announced peer.

We decided not to change it.

- The returned value is simpler.
- We force the initial peer to change so there is no confusion about what was the final announced peer.
@josecelano josecelano self-assigned this Feb 20, 2025
@josecelano josecelano added Documentation Improves Instructions, Guides, and Notices - Developer - Torrust Improvement Experience labels Feb 20, 2025
@josecelano josecelano linked an issue Feb 20, 2025 that may be closed by this pull request
@josecelano
Copy link
Copy Markdown
Member Author

ACK bc95fc4

@codecov
Copy link
Copy Markdown

codecov bot commented Feb 20, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.13%. Comparing base (fd6e902) to head (bc95fc4).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1305   +/-   ##
========================================
  Coverage    81.13%   81.13%           
========================================
  Files          215      215           
  Lines        15279    15275    -4     
  Branches     15279    15275    -4     
========================================
- Hits         12397    12394    -3     
  Misses        2671     2671           
+ Partials       211      210    -1     

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

@josecelano josecelano merged commit 6088528 into torrust:develop Feb 20, 2025
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

- Developer - Torrust Improvement Experience Documentation Improves Instructions, Guides, and Notices

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Overhaul core Tracker: review announce handler (peer mutability)

1 participant