Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Merge bandwidth_accounting module and component #6371

Merged
merged 4 commits into from
Sep 27, 2021

Conversation

drew2a
Copy link
Contributor

@drew2a drew2a commented Sep 27, 2021

This PR contains location changes of files from the bandwidth_accounting module.

All those files are placed in the Bandwidth Accounting Component folder now.

The new structure: https://github.com/drew2a/tribler/tree/feature/merge_bandwidth_accounting/src/tribler-core/tribler_core/components/bandwidth_accounting

image

Addresses #6335

@ichorid
Copy link
Contributor

ichorid commented Sep 27, 2021

Looks nice 👍

@drew2a drew2a force-pushed the feature/merge_bandwidth_accounting branch 4 times, most recently from 90fa5bc to f3c5fbe Compare September 27, 2021 13:44
@drew2a drew2a marked this pull request as ready for review September 27, 2021 13:58
@drew2a drew2a requested review from a team, xoriole and devos50 and removed request for a team September 27, 2021 13:58
devos50
devos50 previously approved these changes Sep 27, 2021
@drew2a drew2a force-pushed the feature/merge_bandwidth_accounting branch from 684c567 to 96a5d23 Compare September 27, 2021 17:00
ichorid
ichorid previously approved these changes Sep 27, 2021
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@drew2a drew2a merged commit cb8e5ef into Tribler:main Sep 27, 2021
@drew2a drew2a deleted the feature/merge_bandwidth_accounting branch September 27, 2021 18:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants