Skip to content

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

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Conversation

josecelano
Copy link
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
Copy link

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
Member Author

ACK 6639c98

@josecelano josecelano merged commit efe7e98 into torrust:develop Feb 12, 2025
23 checks passed
# for free to join this conversation on GitHub. Already have an account? # 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