Skip to content

Overhaul core Tracker: review public methods for tracker-core package#1259

Merged
josecelano merged 1 commit intotorrust:developfrom
josecelano:1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package
Feb 12, 2025
Merged

Overhaul core Tracker: review public methods for tracker-core package#1259
josecelano merged 1 commit intotorrust:developfrom
josecelano:1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package

Conversation

@josecelano
Copy link
Copy Markdown
Member

Overhaul core Tracker: review public methods for tracker-core package.

@josecelano josecelano self-assigned this Feb 11, 2025
@josecelano josecelano requested a review from da2ce7 February 11, 2025 15:30
@josecelano josecelano added Code Cleanup / Refactoring Tidying and Making Neat - Developer - Torrust Improvement Experience labels Feb 11, 2025
@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from abc3d97 to b279628 Compare February 11, 2025 15:30
@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from b279628 to bbb2cd8 Compare February 11, 2025 16:06
@codecov
Copy link
Copy Markdown

codecov bot commented Feb 11, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 81.04%. Comparing base (ea2d73d) to head (6639c98).
Report is 2 commits behind head on develop.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop    #1259   +/-   ##
========================================
  Coverage    81.03%   81.04%           
========================================
  Files          206      207    +1     
  Lines        14748    14753    +5     
  Branches     14748    14753    +5     
========================================
+ Hits         11951    11956    +5     
+ Misses        2599     2595    -4     
- Partials       198      202    +4     

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

@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from bbb2cd8 to 139453d Compare February 11, 2025 16:44
@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from 139453d to 62e9eba Compare February 11, 2025 17:12
@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from 62e9eba to 53b94a3 Compare February 11, 2025 17:14
@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from 53b94a3 to 9337817 Compare February 11, 2025 17:23
…ble.

Limit the exposed funtionality for pacakges. Specially the new
`tracker-core` package which has not been published yet.
@josecelano josecelano force-pushed the 1258-overhaul-core-tracker-review-public-methods-for-tracker-core-package branch from 9337817 to 6639c98 Compare February 12, 2025 07:16
@josecelano josecelano marked this pull request as ready for review February 12, 2025 07:17
@josecelano
Copy link
Copy Markdown
Member Author

ACK 6639c98

@josecelano josecelano merged commit efe7e98 into torrust:develop Feb 12, 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 Code Cleanup / Refactoring Tidying and Making Neat

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Overhaul core Tracker: review public methods for tracker-core package

1 participant